about summary refs log tree commit homepage
path: root/t/nntp.t
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2016-08-14 10:21:09 +0000
committerEric Wong <e@80x24.org>2016-08-14 10:51:02 +0000
commit9d1e5fadd7d18f4c96ab0509d673040e34225a04 (patch)
treedd89df05390a2ca2e06448ec0acdbf741729d36e /t/nntp.t
parent1d0ce406b75fc174bee51e77efe5f10c61fb6098 (diff)
downloadpublic-inbox-9d1e5fadd7d18f4c96ab0509d673040e34225a04.tar.gz
Based on reading RFC 3986, it seems '@', ':', '!', '$', '&',
"'", '; '(', ')', '*', '+', ',', ';', '=' are all allowed
in path-absolute where we have the Message-ID.

In any case, it seems '@' is fairly common in path components
nowadays and too common in Message-IDs.
Diffstat (limited to 't/nntp.t')
-rw-r--r--t/nntp.t4
1 files changed, 2 insertions, 2 deletions
diff --git a/t/nntp.t b/t/nntp.t
index de07abb0..7500d6b9 100644
--- a/t/nntp.t
+++ b/t/nntp.t
@@ -112,7 +112,7 @@ use_ok 'PublicInbox::Inbox';
         PublicInbox::NNTP::set_nntp_headers($mime->header_obj, $ng, 1, $mid);
         is_deeply([ $mime->header('Message-ID') ], [ "<$mid>" ],
                 'Message-ID unchanged');
-        is_deeply([ $mime->header('Archived-At') ], [ "<${u}a%40b/>" ],
+        is_deeply([ $mime->header('Archived-At') ], [ "<${u}a\@b/>" ],
                 'Archived-At: set');
         is_deeply([ $mime->header('List-Archive') ], [ "<$u>" ],
                 'List-Archive: set');
@@ -128,7 +128,7 @@ use_ok 'PublicInbox::Inbox';
         is_deeply([ $mime->header('Message-ID') ], [ "<$mid>" ],
                 'Message-ID unchanged');
         is_deeply([ $mime->header('Archived-At') ],
-                [ "<${u}a%40b/>", '<http://mirror.example.com/m/a%40b/>' ],
+                [ "<${u}a\@b/>", '<http://mirror.example.com/m/a@b/>' ],
                 'Archived-At: appended');
         is_deeply([ $mime->header('Xref') ], [ 'example.com test:2' ],
                 'Old Xref: clobbered');