unofficial mirror of meta@public-inbox.org
 help / color / mirror / Atom feed
* [PATCH] mid2path: clean MID of angle brackets '<>'
@ 2015-08-30  1:34 Eric Wong
  0 siblings, 0 replies; only message in thread
From: Eric Wong @ 2015-08-30  1:34 UTC (permalink / raw)
  To: meta

We screwed up and needed to fix URL generation with '<>'
in them.  Regardless, users may attempt to copy and paste
URLs with '<>' in them, do not punish them for that.
---
 lib/PublicInbox/MID.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/PublicInbox/MID.pm b/lib/PublicInbox/MID.pm
index 8ca3c57..1b03fdc 100644
--- a/lib/PublicInbox/MID.pm
+++ b/lib/PublicInbox/MID.pm
@@ -39,7 +39,7 @@ sub mid2path {
 
 	unless (defined $x38) {
 		# compatibility with old links (or short Message-IDs :)
-		$mid = sha1_hex($mid);
+		$mid = sha1_hex(mid_clean($mid));
 		($x2, $x38) = ($mid =~ /\A([a-f0-9]{2})([a-f0-9]{38})\z/);
 	}
 	"$x2/$x38";
-- 
EW


^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2015-08-30  1:34 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-30  1:34 [PATCH] mid2path: clean MID of angle brackets '<>' Eric Wong

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).