From 4d02ed7d60ecff963d3026187a0357e359fd2981 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Sat, 2 Jul 2016 07:56:36 +0000 Subject: www: use PSGI env directly More work on on the Plack::Request/CGI.pm removal front, No need to access the PSGI env through an extra hash lookup. --- lib/PublicInbox/SearchView.pm | 2 +- lib/PublicInbox/WWW.pm | 9 ++++----- 2 files changed, 5 insertions(+), 6 deletions(-) (limited to 'lib/PublicInbox') diff --git a/lib/PublicInbox/SearchView.pm b/lib/PublicInbox/SearchView.pm index ce1eff1e..15bb8232 100644 --- a/lib/PublicInbox/SearchView.pm +++ b/lib/PublicInbox/SearchView.pm @@ -86,7 +86,7 @@ sub mset_summary { sub err_txt { my ($ctx, $err) = @_; my $u = '//xapian.org/docs/queryparser.html'; - $u = PublicInbox::Hval::prurl($ctx->{cgi}->{env}, $u); + $u = PublicInbox::Hval::prurl($ctx->{env}, $u); $err =~ s/^\s*Exception:\s*//; # bad word to show users :P $err = ascii_html($err); "\nBad query: $err\n" . diff --git a/lib/PublicInbox/WWW.pm b/lib/PublicInbox/WWW.pm index 1e23c439..940e1c51 100644 --- a/lib/PublicInbox/WWW.pm +++ b/lib/PublicInbox/WWW.pm @@ -152,7 +152,7 @@ sub invalid_inbox { $ctx->{git_dir} = $obj->{mainrepo}; $ctx->{git} = $obj->git; # for PublicInbox::HTTP::weaken_task: - $ctx->{cgi}->{env}->{'pi-httpd.inbox'} = $obj; + $ctx->{env}->{'pi-httpd.inbox'} = $obj; $ctx->{-inbox} = $obj; $ctx->{inbox} = $inbox; return; @@ -162,7 +162,7 @@ sub invalid_inbox { # generation and link things intended for nntp:// to https?://, # so try to infer links and redirect them to the appropriate # list URL. - $self->news_www->call($ctx->{cgi}->{env}); + $self->news_www->call($ctx->{env}); } # returns undef if valid, array ref response if invalid @@ -284,7 +284,7 @@ sub footer { $ctx->{footer} = join("\n", '- ' . $desc, "A {cgi}->{env}, PI_URL) . + PublicInbox::Hval::prurl($ctx->{env}, PI_URL) . '">public-inbox, ' . 'anybody may post in plain-text (not HTML):', $addr, @@ -388,13 +388,12 @@ sub legacy_redirects { } elsif ($path_info =~ m!$INBOX_RE/(\S+/\S+)/f\z!o) { r301($ctx, $1, $2); } else { - $self->news_www->call($ctx->{cgi}->{env}); + $self->news_www->call($ctx->{env}); } } sub r301 { my ($ctx, $inbox, $mid, $suffix) = @_; - my $cgi = $ctx->{cgi}; my $obj = $ctx->{-inbox}; unless ($obj) { my $r404 = invalid_inbox($ctx->{www}, $ctx, $inbox); -- cgit v1.2.3-24-ge0c7