diff options
author | Eric Wong <e@80x24.org> | 2014-04-15 07:10:17 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2014-04-15 07:13:43 +0000 |
commit | 3f9410e7a2663dc3f341f82687bef3e9dbea336a (patch) | |
tree | 471fa11847f42e0e0df5b2ef52df93687cf74eb9 /public-inbox.cgi | |
parent | d95eccbe97f1a48b76c8a80fc81d88a291676cdc (diff) | |
download | public-inbox-3f9410e7a2663dc3f341f82687bef3e9dbea336a.tar.gz |
CGI mounts should probably handle this internally. We're reverting this since it adds too much potential for abuse with fake/extra prefixes in the URL. We also need to reorder our redirect handling as a result. This reverts commit c394de9f2c91c2c5ed1f7832a5a7cc0206120b7f.
Diffstat (limited to 'public-inbox.cgi')
-rwxr-xr-x | public-inbox.cgi | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/public-inbox.cgi b/public-inbox.cgi index 1fb96e75..9d60d1fc 100755 --- a/public-inbox.cgi +++ b/public-inbox.cgi @@ -17,7 +17,7 @@ use Encode qw(decode_utf8); use PublicInbox::Config; use URI::Escape qw(uri_escape uri_unescape); use Digest::SHA qw(sha1_hex); -our $LISTNAME_RE = qr!\A(?:/.*?)?/([\w\.\-]+)!; +our $LISTNAME_RE = qr!\A/([\w\.\-]+)!; our $pi_config; BEGIN { $pi_config = PublicInbox::Config->new; @@ -62,7 +62,9 @@ sub main { # top-level indices and feeds if ($path_info eq "/") { r404(); - } elsif ($path_info =~ m!$LISTNAME_RE/(?:index\.html)?\z!o) { + } elsif ($path_info =~ m!$LISTNAME_RE\z!o) { + invalid_list(\%ctx, $1) || redirect_list_index(\%ctx, $cgi); + } elsif ($path_info =~ m!$LISTNAME_RE(?:/|/index\.html)?\z!o) { invalid_list(\%ctx, $1) || get_index(\%ctx, $cgi, 1); } elsif ($path_info =~ m!$LISTNAME_RE/(?:all\.html)?\z!o) { invalid_list(\%ctx, $1) || get_index(\%ctx, $cgi, 0); @@ -85,8 +87,6 @@ sub main { } elsif ($path_info =~ m!$LISTNAME_RE/f/(\S+)\z!o) { redirect_mid_html($cgi, $1, $2); - } elsif ($path_info =~ m!$LISTNAME_RE\z!o) { - invalid_list(\%ctx, $1) || redirect_list_index(\%ctx, $cgi); } else { r404(); } |