diff options
author | Eric Wong <e@80x24.org> | 2022-09-10 20:10:23 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2022-09-11 19:25:46 +0000 |
commit | f5e2daa70e0a1a75d1220d84193f3eeebb799c38 (patch) | |
tree | f5a5a8bcf8982ab5b025f8070011e099eeb55cab /lib/PublicInbox/Feed.pm | |
parent | 12900fecdc4c7f577eb8266a2334d74f72fd2518 (diff) | |
download | public-inbox-f5e2daa70e0a1a75d1220d84193f3eeebb799c38.tar.gz |
For redundant messages sharing Message-IDs, the link to solver (/$INBOX/$OID/s/) was going up too many levels for /$INBOX/$MSGID/ when there were multiple messages sharing the same $MSGID. Unfortunately, redundant messages are common with /all/ due to signature trailers. So dynamically assigning {-spfx} is tricky and error prone from counting `/'. So simplify the code a bit by setting {-spfx} once per HTTP request, instead of every single message.
Diffstat (limited to 'lib/PublicInbox/Feed.pm')
-rw-r--r-- | lib/PublicInbox/Feed.pm | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/PublicInbox/Feed.pm b/lib/PublicInbox/Feed.pm index affe0fb6..de1e7dfe 100644 --- a/lib/PublicInbox/Feed.pm +++ b/lib/PublicInbox/Feed.pm @@ -70,6 +70,7 @@ sub new_html { } $ctx->{-html_tip} = '<pre>'; $ctx->{-upfx} = ''; + $ctx->{-spfx} = '' if $ctx->{ibx}->{coderepo}; $ctx->{-hr} = 1; PublicInbox::WwwStream::aresponse($ctx, \&new_html_i); } |