From: Eric Wong <e@80x24.org>
Date: Tue, 17 May 2016 05:39:06 +0000 (+0000)
Subject: view: escape Message-ID for "next" link
X-Git-Tag: v1.0.0~536
X-Git-Url: http://www.git.stargrave.org/?a=commitdiff_plain;h=c1e768a619b01498ec391431e5fa31221aca89a7;p=public-inbox.git

view: escape Message-ID for "next" link

Oops, we need to escape Message-IDs since they can contain
bad characters such as '%' in them.  '@' actually seems fine
and does not need to be escaped; however, but we've been
doing it forever.
---

diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm
index d559cae4..3fa7cfc2 100644
--- a/lib/PublicInbox/View.pm
+++ b/lib/PublicInbox/View.pm
@@ -518,7 +518,8 @@ sub html_footer {
 			$irt = ' ' x length('parent ');
 		}
 		if ($next) {
-			$irt .= "<a\nhref=\"$upfx$next/\"\nrel=next>next</a> ";
+			my $n = PublicInbox::Hval->new_msgid($next)->as_href;
+			$irt .= "<a\nhref=\"$upfx$n/\"\nrel=next>next</a> ";
 		} else {
 			$irt .= ' ' x length('next ');
 		}