about summary refs log tree commit homepage
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2017-02-14 23:19:34 +0000
committerEric Wong <e@80x24.org>2017-02-14 23:19:34 +0000
commit36c89ffaf34c6fe50bd8c49f44163dbb85680a43 (patch)
tree2985b6100c477edfc1d785389d16f29b72b1e191
parent7c598ac6a81140e1b6d231809de2f4e7e296984b (diff)
downloadpublic-inbox-36c89ffaf34c6fe50bd8c49f44163dbb85680a43.tar.gz
PSGI specs already require PATH_INFO to be unescaped.

Followup-to: commit 364de65f8a6b5729027cb70228312a141430122f
("www: do not unescape PATH_INFO twice")
-rw-r--r--lib/PublicInbox/Repobrowse.pm4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/PublicInbox/Repobrowse.pm b/lib/PublicInbox/Repobrowse.pm
index 87e12278..c16f10fd 100644
--- a/lib/PublicInbox/Repobrowse.pm
+++ b/lib/PublicInbox/Repobrowse.pm
@@ -20,7 +20,7 @@
 package PublicInbox::Repobrowse;
 use strict;
 use warnings;
-use URI::Escape qw(uri_escape_utf8 uri_unescape);
+use URI::Escape qw(uri_escape_utf8);
 use PublicInbox::RepoConfig;
 
 my %CMD = map { lc($_) => $_ } qw(Log Commit Tree Patch Blob Plain Tag Atom
@@ -92,7 +92,7 @@ sub call {
         # URL syntax: / repo [ / cmd [ / path ] ]
         # cmd: log | commit | diff | tree | view | blob | snapshot
         # repo and path (@extra) may both contain '/'
-        my $path_info = uri_unescape($env->{PATH_INFO});
+        my $path_info = $env->{PATH_INFO};
         my (undef, $repo_path, @extra) = split(m{/+}, $path_info, -1);
 
         return $self->root_index($self) unless length($repo_path);