diff options
| author | Eric Wong <e@80x24.org> | 2020-12-09 23:33:00 +0000 |
|---|---|---|
| committer | Eric Wong <e@80x24.org> | 2020-12-10 06:39:29 +0000 |
| commit | db7c206a4e2c71d08dd286d6dfa92431976e9693 (patch) | |
| tree | a9f044f2d7e77afd8cd5ab55bfe429fde7c371f9 /lib/PublicInbox/NNTP.pm | |
| parent | 443ab96a3b329556e26a57b02c7383d5faf536b3 (diff) | |
| download | public-inbox-db7c206a4e2c71d08dd286d6dfa92431976e9693.tar.gz | |
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')
| -rw-r--r-- | lib/PublicInbox/NNTP.pm | 5 |
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/>"); |