From 8d446e962b7fb66c235db59a819aca3c34acdcb0 Mon Sep 17 00:00:00 2001 From: Aurélien Bompard Date: Tue, 30 Jun 2015 15:56:40 +0200 Subject: According to RFC 2369, the URL should be between brackets --- src/mailman/app/docs/pipelines.rst | 2 +- src/mailman/handlers/rfc_2369.py | 2 +- src/mailman/handlers/tests/test_rfc_2369.py | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mailman/app/docs/pipelines.rst b/src/mailman/app/docs/pipelines.rst index dfdc6d70c..eb59fcfc2 100644 --- a/src/mailman/app/docs/pipelines.rst +++ b/src/mailman/app/docs/pipelines.rst @@ -49,7 +49,7 @@ etc. Precedence: list Subject: [Test] My first post List-Id: - Archived-At: http://lists.example.com/.../4CMWUN6BHVCMHMDAOSJZ2Q72G5M32MWB + Archived-At: List-Archive: List-Help: List-Post: diff --git a/src/mailman/handlers/rfc_2369.py b/src/mailman/handlers/rfc_2369.py index 8cb586abf..ab7fc76ea 100644 --- a/src/mailman/handlers/rfc_2369.py +++ b/src/mailman/handlers/rfc_2369.py @@ -91,7 +91,7 @@ def process(mlist, msg, msgdata): '<{}>'.format(archiver_url))) permalink = archiver.system_archiver.permalink(mlist, msg) if permalink is not None: - headers.append(('Archived-At', permalink)) + headers.append(('Archived-At', '<{}>'.format(permalink))) # XXX RFC 2369 also defines a List-Owner header which we are not currently # supporting, but should. # diff --git a/src/mailman/handlers/tests/test_rfc_2369.py b/src/mailman/handlers/tests/test_rfc_2369.py index 3ee3a7ca2..b297d04c2 100644 --- a/src/mailman/handlers/tests/test_rfc_2369.py +++ b/src/mailman/handlers/tests/test_rfc_2369.py @@ -97,7 +97,7 @@ Dummy text ['']) self.assertEqual( self._msg.get_all('Archived-At'), - ['http://lists.example.com/4CMWUN6BHVCMHMDAOSJZ2Q72G5M32MWB']) + ['']) def test_prototype_no_url(self): # The prototype archiver is not web-based, it must not return URLs -- cgit v1.3.1