summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBarry Warsaw2014-04-15 10:03:39 -0400
committerBarry Warsaw2014-04-15 10:03:39 -0400
commit2c7c9314d5ff8224273ac62d1096c9aa121bc8e0 (patch)
treefa1e7fb167ebe3cd380d946fcef8f048961c7776
parentee8abcd62038917e0b1baf6f99631f88b464add2 (diff)
downloadmailman-2c7c9314d5ff8224273ac62d1096c9aa121bc8e0.tar.gz
mailman-2c7c9314d5ff8224273ac62d1096c9aa121bc8e0.tar.zst
mailman-2c7c9314d5ff8224273ac62d1096c9aa121bc8e0.zip
-rw-r--r--src/mailman/app/membership.py3
-rw-r--r--src/mailman/app/subscriptions.py5
2 files changed, 0 insertions, 8 deletions
diff --git a/src/mailman/app/membership.py b/src/mailman/app/membership.py
index fdeb6e9c4..4ec6b7878 100644
--- a/src/mailman/app/membership.py
+++ b/src/mailman/app/membership.py
@@ -35,7 +35,6 @@ from mailman.app.notifications import (
from mailman.config import config
from mailman.core.i18n import _
from mailman.email.message import OwnerNotification
-from mailman.interfaces.address import IEmailValidator
from mailman.interfaces.bans import IBanManager
from mailman.interfaces.member import (
MemberRole, MembershipIsBannedError, NotAMemberError, SubscriptionEvent)
@@ -72,8 +71,6 @@ def add_member(mlist, email, display_name, password, delivery_mode, language,
:raises InvalidEmailAddressError: if the email address is not valid.
:raises MembershipIsBannedError: if the membership is not allowed.
"""
- # Let's be extra cautious.
- getUtility(IEmailValidator).validate(email)
# Check to see if the email address is banned.
if IBanManager(mlist).is_banned(email):
raise MembershipIsBannedError(mlist, email)
diff --git a/src/mailman/app/subscriptions.py b/src/mailman/app/subscriptions.py
index d80069ba5..b2560beb5 100644
--- a/src/mailman/app/subscriptions.py
+++ b/src/mailman/app/subscriptions.py
@@ -37,7 +37,6 @@ from mailman.app.membership import add_member, delete_member
from mailman.config import config
from mailman.core.constants import system_preferences
from mailman.database.transaction import dbconnection
-from mailman.interfaces.address import IEmailValidator
from mailman.interfaces.listmanager import (
IListManager, ListDeletingEvent, NoSuchListError)
from mailman.interfaces.member import DeliveryMode, MemberRole
@@ -152,10 +151,6 @@ class SubscriptionService:
raise NoSuchListError(list_id)
# Is the subscriber an email address or user id?
if isinstance(subscriber, basestring):
- # It's an email address, so we'll want a real name. Make sure
- # it's a valid email address, and let InvalidEmailAddressError
- # propagate up.
- getUtility(IEmailValidator).validate(subscriber)
if display_name is None:
display_name, at, domain = subscriber.partition('@')
# Because we want to keep the REST API simple, there is no