diff options
author | Ruben Pollan <meskio@sindominio.net> | 2015-10-28 18:52:34 +0100 |
---|---|---|
committer | Ruben Pollan <meskio@sindominio.net> | 2015-11-23 17:13:33 +0100 |
commit | 6fd862abc9e8711bd4aa117697c5df85cf6f560a (patch) | |
tree | 6a31b4ecd4bff0333c16cf9377e29c5e869ec856 | |
parent | ebc748d6dbe366d82e601c94f4191718b310bfae (diff) |
[feat] add '@deliver.local' to uuid
- Resolves: #5959
-rw-r--r-- | changes/next-changelog.txt | 29 | ||||
-rw-r--r-- | src/leap/mx/alias_resolver.py | 3 | ||||
-rw-r--r-- | src/leap/mx/mail_receiver.py | 4 | ||||
-rw-r--r-- | src/leap/mx/tests/test_mail_receiver.py | 7 |
4 files changed, 37 insertions, 6 deletions
diff --git a/changes/next-changelog.txt b/changes/next-changelog.txt new file mode 100644 index 0000000..c7ba3b8 --- /dev/null +++ b/changes/next-changelog.txt @@ -0,0 +1,29 @@ +0.8.0 - xxx ++++++++++++++++++++++++++++++++ + +Please add lines to this file, they will be moved to the CHANGELOG.rst during +the next release. + +There are two template lines for each category, use them as reference. + +I've added a new category `Misc` so we can track doc/style/packaging stuff. + +Features +~~~~~~~~ +- `#5959 <https://leap.se/code/issues/5959>`_: Make alias resolver to return *uuid@deliver.local* +- `#1234 <https://leap.se/code/issues/1234>`_: Description of the new feature corresponding with issue #1234. +- New feature without related issue number. + +Bugfixes +~~~~~~~~ +- `#1235 <https://leap.se/code/issues/1235>`_: Description for the fixed stuff corresponding with issue #1235. +- Bugfix without related issue number. + +Misc +~~~~ +- `#1236 <https://leap.se/code/issues/1236>`_: Description of the new feature corresponding with issue #1236. +- Some change without issue number. + +Known Issues +~~~~~~~~~~~~ +- `#1236 <https://leap.se/code/issues/1236>`_: Description of the known issue corresponding with issue #1236. diff --git a/src/leap/mx/alias_resolver.py b/src/leap/mx/alias_resolver.py index bf7a58b..a5b5107 100644 --- a/src/leap/mx/alias_resolver.py +++ b/src/leap/mx/alias_resolver.py @@ -1,7 +1,7 @@ #!/usr/bin/env python # -*- encoding: utf-8 -*- # alias_resolver.py -# Copyright (C) 2013 LEAP +# Copyright (C) 2013, 2015 LEAP # # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -60,6 +60,7 @@ class LEAPPostfixTCPMapAliasServer(postfix.PostfixTCPMapServer): TCP_MAP_CODE_PERMANENT_FAILURE, postfix.quote("NOT FOUND SRY")) else: + uuid += "@deliver.local" # properly encode uuid, otherwise twisted complains when replying if isinstance(uuid, unicode): uuid = uuid.encode("utf8") diff --git a/src/leap/mx/mail_receiver.py b/src/leap/mx/mail_receiver.py index a9344c3..ea13658 100644 --- a/src/leap/mx/mail_receiver.py +++ b/src/leap/mx/mail_receiver.py @@ -1,7 +1,7 @@ #!/usr/bin/env python # -*- encoding: utf-8 -*- # mail_receiver.py -# Copyright (C) 2013 LEAP +# Copyright (C) 2013, 2015 LEAP # # This program is free software: you can redistribute it and/or modify # it under the terms of the GNU General Public License as published by @@ -269,7 +269,7 @@ class MailReceiver(Service): return None final_address = delivereds.pop(0) _, addr = email.utils.parseaddr(final_address) - uuid, _ = addr.split("@") + uuid = addr.split("@")[0] return uuid @defer.inlineCallbacks diff --git a/src/leap/mx/tests/test_mail_receiver.py b/src/leap/mx/tests/test_mail_receiver.py index 3eeafbd..e72cb1a 100644 --- a/src/leap/mx/tests/test_mail_receiver.py +++ b/src/leap/mx/tests/test_mail_receiver.py @@ -37,6 +37,7 @@ from leap.mx.mail_receiver import MailReceiver BOUNCE_ADDRESS = "bounce@leap.se" BOUNCE_SUBJECT = "bounce subject" ADDRESS = "leap@leap.se" +UUID = "13d5203bdd09be1e638bdb1d315251cb" class MailReceiverTestCase(unittest.TestCase): @@ -78,8 +79,7 @@ class MailReceiverTestCase(unittest.TestCase): def test_single_mail(self): msg, path = self.addMail("foo bar") uuid, doc = yield self.defer_put_doc - orig_uuid = ADDRESS.split('@')[0] - self.assertEqual(uuid, orig_uuid) + self.assertEqual(uuid, UUID) decmsg = self.decryptDoc(doc) self.assertEqual(msg, decmsg) self.assertFalse(os.path.exists(path)) @@ -101,7 +101,8 @@ class MailReceiverTestCase(unittest.TestCase): frm="someone@domain.org", subject="sent subject"): msg = Message() msg.add_header("To", to) - msg.add_header("Delivered-To", to) + msg.add_header( + "Delivered-To", UUID + "@deliver.local") msg.add_header("From", frm) msg.add_header("Subject", subject) msg.set_payload(body) |