diff options
| author | black-perl ankprashar@gmail.com | 2015-03-04 14:38:19 +0530 |
|---|---|---|
| committer | black-perl ankprashar@gmail.com | 2015-03-04 14:38:19 +0530 |
| commit | 4044a657df9b1c332954267b42f8ffe7820e45f0 (patch) | |
| tree | 359f1fdbfd182fde3f5d351106564d6e63001da2 /src | |
| parent | 73f7fc7f54a4ded95ec31303baed42ca956f82e0 (diff) | |
| download | mailman-4044a657df9b1c332954267b42f8ffe7820e45f0.tar.gz mailman-4044a657df9b1c332954267b42f8ffe7820e45f0.tar.zst mailman-4044a657df9b1c332954267b42f8ffe7820e45f0.zip | |
Fixed the issue #1425359 and wrote corresponding tests.
Diffstat (limited to 'src')
| -rw-r--r-- | src/mailman/app/membership.py | 2 | ||||
| -rw-r--r-- | src/mailman/rest/tests/test_membership.py | 9 |
2 files changed, 10 insertions, 1 deletions
diff --git a/src/mailman/app/membership.py b/src/mailman/app/membership.py index c50169a7c..0a3b4b352 100644 --- a/src/mailman/app/membership.py +++ b/src/mailman/app/membership.py @@ -98,7 +98,7 @@ def add_member(mlist, email, display_name, password, delivery_mode, language, else: # The user exists and is linked to the address. for address in user.addresses: - if address.email == email: + if address.email == email.lower(): break else: raise AssertionError( diff --git a/src/mailman/rest/tests/test_membership.py b/src/mailman/rest/tests/test_membership.py index e1bff833b..a23f49763 100644 --- a/src/mailman/rest/tests/test_membership.py +++ b/src/mailman/rest/tests/test_membership.py @@ -98,6 +98,15 @@ class TestMembership(unittest.TestCase): self.assertEqual(cm.exception.code, 409) self.assertEqual(cm.exception.reason, b'Member already subscribed') + # Member subscription using case sensitive email + with self.assertRaises(HTTPError) as cm: + call_api('http://localhost:9001/3.0/members', { + 'list_id': 'test.example.com', + 'subscriber': 'ANNE@example.com', + }) + self.assertEqual(cm.exception.code, 409) + self.assertEqual(cm.exception.reason, b'Member already subscribed') + def test_join_with_invalid_delivery_mode(self): with self.assertRaises(HTTPError) as cm: call_api('http://localhost:9001/3.0/members', { |