diff options
author | Eric Wong <e@80x24.org> | 2020-12-10 22:38:47 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2020-12-11 03:40:20 +0000 |
commit | 1bf653ad139bf7bb3d853ab0b5eae3eaa1b13a95 (patch) | |
tree | 389bacf02a9e80bcb445c59fafad8159f63f3610 /t/nntp.t | |
parent | ac4f951e03eaf635c8dd20a6f42bfc71b0011412 (diff) | |
download | public-inbox-1bf653ad139bf7bb3d853ab0b5eae3eaa1b13a95.tar.gz |
These headers can conflict with headers in the DKIM signature; and parsing the DKIM-Signature header to determine whether or not we can safely add a header would be more code and CPU cycles. Since IMAP seems fine without these headers (and JMAP will likely be, too), there's likely no need to continue appending these to every message. Nowadays, developers seem sufficiently trained to use URLs with Message-IDs in them. So drop the headers and save some cycles and bandwidth all around.
Diffstat (limited to 't/nntp.t')
-rw-r--r-- | t/nntp.t | 9 |
1 files changed, 0 insertions, 9 deletions
@@ -121,12 +121,6 @@ use PublicInbox::Config; PublicInbox::NNTP::set_nntp_headers($hdr, $smsg); is_deeply([ $mime->header('Message-ID') ], [ "<$mid>" ], 'Message-ID unchanged'); - is_deeply([ $mime->header('Archived-At') ], [ "<${u}a\@b/>" ], - 'Archived-At: set'); - is_deeply([ $mime->header('List-Archive') ], [ "<$u>" ], - 'List-Archive: set'); - is_deeply([ $mime->header('List-Post') ], [ '<mailto:a@example.com>' ], - 'List-Post: set'); is_deeply([ $mime->header('Newsgroups') ], [ 'test' ], 'Newsgroups: set'); is_deeply([ $mime->header('Xref') ], [ 'example.com test:1' ], @@ -137,9 +131,6 @@ use PublicInbox::Config; PublicInbox::NNTP::set_nntp_headers($hdr, $smsg); is_deeply([ $mime->header('Message-ID') ], [ "<$mid>" ], 'Message-ID unchanged'); - is_deeply([ $mime->header('Archived-At') ], - [ "<${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'); } |