about summary refs log tree commit homepage
path: root/lib/PublicInbox/Mbox.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2016-04-12 21:25:05 +0000
committerEric Wong <e@80x24.org>2016-04-12 21:25:24 +0000
commitae43d0cf268512ce3dff6c8af3e4f686b2a011e2 (patch)
tree95b76520ba1388b3995197b6cdb4944f0c1e00a8 /lib/PublicInbox/Mbox.pm
parent56164afc203489920ca9ef4b7407b39625acfa9f (diff)
downloadpublic-inbox-ae43d0cf268512ce3dff6c8af3e4f686b2a011e2.tar.gz
When serving archives, it's more robust to keep existing
archive links in one server goes down.
Diffstat (limited to 'lib/PublicInbox/Mbox.pm')
-rw-r--r--lib/PublicInbox/Mbox.pm20
1 files changed, 15 insertions, 5 deletions
diff --git a/lib/PublicInbox/Mbox.pm b/lib/PublicInbox/Mbox.pm
index 9f20580e..ffa6fa47 100644
--- a/lib/PublicInbox/Mbox.pm
+++ b/lib/PublicInbox/Mbox.pm
@@ -48,11 +48,21 @@ sub emit_msg {
         my $base = $feed_opts->{url};
         my $mid = mid_clean($header_obj->header('Message-ID'));
         $mid = uri_escape_utf8($mid);
-        my @archived_at = $header_obj->header('Archived-At');
-        push @archived_at, "<$base$mid/>";
-        $header_obj->header_set('Archived-At', @archived_at);
-        $header_obj->header_set('List-Archive', "<$base>");
-        $header_obj->header_set('List-Post', "<mailto:$feed_opts->{id_addr}>");
+        my %append = (
+                'Archived-At' => "<$base$mid/>",
+                'List-Archive' => "<$base>",
+                'List-Post' => "<mailto:$feed_opts->{id_addr}>",
+        );
+        while (my ($k, $v) = each %append) {
+                my @v = $header_obj->header($k);
+                foreach (@v) {
+                        if ($v eq $_) {
+                                $v = undef;
+                                last;
+                        }
+                }
+                $header_obj->header_set($k, @v, $v) if defined $v;
+        }
 
         my $buf = $header_obj->as_string;
         unless ($buf =~ /\AFrom /) {