about summary refs log tree commit homepage
path: root/lib/PublicInbox/NNTP.pm
diff options
authorEric Wong <e@80x24.org>2020-12-09 23:33:00 +0000
committerEric Wong <e@80x24.org>2020-12-10 06:39:29 +0000
commitdb7c206a4e2c71d08dd286d6dfa92431976e9693 (patch)
treea9f044f2d7e77afd8cd5ab55bfe429fde7c371f9 /lib/PublicInbox/NNTP.pm
parent443ab96a3b329556e26a57b02c7383d5faf536b3 (diff)
www+nntp: deal with lack of addresses for ->ALL
Since extindex is an amalgamation of several inboxes, discerning
an appropriate address for List-Post: would be expensive and
most likely unnecessary.  Some legacy/historical inboxes may
have no active address, either, so don't attempt to set the
List-Post header if no addresses are configured.
Diffstat (limited to 'lib/PublicInbox/NNTP.pm')
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/PublicInbox/NNTP.pm b/lib/PublicInbox/NNTP.pm
index 0b43cdbc..097fdb84 100644
--- a/lib/PublicInbox/NNTP.pm
+++ b/lib/PublicInbox/NNTP.pm
@@ -487,8 +487,9 @@ sub set_nntp_headers ($$) {
         # *something* here is required for leafnode, try to follow
         # RFC 5536 3.1.5...
         $hdr->header_set('Path', $server_name . '!not-for-mail');
-        header_append($hdr, 'List-Post', "<mailto:$ibx->{-primary_address}>");
+        if (my $post_addr = $ibx->{-primary_address}) {
+                header_append($hdr, 'List-Post', "<mailto:$post_addr>");
+        }
         if (my $url = $ibx->base_url) {
                 $mid = mid_escape($mid);
                 header_append($hdr, 'Archived-At', "<$url$mid/>");