Merged in feature/VBV-635-invalid-emails (pull request #273)
Handle invalid recipient email addresses Approved-by: Daniel Egger
This commit is contained in:
commit
7c89751850
|
|
@ -3,6 +3,8 @@ from __future__ import annotations
|
||||||
from typing import TYPE_CHECKING
|
from typing import TYPE_CHECKING
|
||||||
|
|
||||||
import structlog
|
import structlog
|
||||||
|
from django.core.exceptions import ValidationError
|
||||||
|
from django.core.validators import validate_email
|
||||||
from django.db.models import Model
|
from django.db.models import Model
|
||||||
from notifications.signals import notify
|
from notifications.signals import notify
|
||||||
|
|
||||||
|
|
@ -281,6 +283,13 @@ class NotificationService:
|
||||||
template_data=template_data,
|
template_data=template_data,
|
||||||
)
|
)
|
||||||
emailed = False
|
emailed = False
|
||||||
|
|
||||||
|
try:
|
||||||
|
validate_email(recipient.email)
|
||||||
|
except ValidationError:
|
||||||
|
log.info("Recipient email is invalid")
|
||||||
|
return f"{notification_identifier}_invalid_email"
|
||||||
|
|
||||||
try:
|
try:
|
||||||
notification = NotificationService._find_duplicate_notification(
|
notification = NotificationService._find_duplicate_notification(
|
||||||
recipient=recipient,
|
recipient=recipient,
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue