X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FCatalyst%2FEngine%2FCGI.pm;h=950cc2d709cdddd972c6b47d8f421001677baf5f;hb=c825a86ca867b6b804aa8636728c3d3702ac9f04;hp=82ea2cac000c1e8a4c8411694464574ba677f304;hpb=fbcc39ad23f2bbecf5d84c9ba581e6af86fcd460;p=catagits%2FCatalyst-Runtime.git diff --git a/lib/Catalyst/Engine/CGI.pm b/lib/Catalyst/Engine/CGI.pm index 82ea2ca..950cc2d 100644 --- a/lib/Catalyst/Engine/CGI.pm +++ b/lib/Catalyst/Engine/CGI.pm @@ -4,7 +4,6 @@ use strict; use base 'Catalyst::Engine'; use NEXT; use URI; -use URI::Query; =head1 NAME @@ -54,21 +53,21 @@ sub finalize_headers { sub prepare_connection { my ( $self, $c ) = @_; - + $c->request->address( $ENV{REMOTE_ADDR} ); - - PROXY_CHECK: + + PROXY_CHECK: { unless ( $c->config->{using_frontend_proxy} ) { last PROXY_CHECK if $ENV{REMOTE_ADDR} ne '127.0.0.1'; last PROXY_CHECK if $c->config->{ignore_frontend_proxy}; } last PROXY_CHECK unless $ENV{HTTP_X_FORWARDED_FOR}; - + # If we are running as a backend server, the user will always appear # as 127.0.0.1. Select the most recent upstream IP (last in the list) my ($ip) = $ENV{HTTP_X_FORWARDED_FOR} =~ /([^,\s]+)$/; - $c->request->address( $ip ); + $c->request->address($ip); } $c->request->hostname( $ENV{REMOTE_HOST} ); @@ -107,13 +106,13 @@ sub prepare_headers { sub prepare_path { my ( $self, $c ) = @_; - my $scheme = $c->request->secure ? 'https' : 'http'; + my $scheme = $c->request->secure ? 'https' : 'http'; my $host = $ENV{HTTP_HOST} || $ENV{SERVER_NAME}; my $port = $ENV{SERVER_PORT} || 80; my $base_path = $ENV{SCRIPT_NAME} || '/'; - + # If we are running as a backend proxy, get the true hostname - PROXY_CHECK: + PROXY_CHECK: { unless ( $c->config->{using_frontend_proxy} ) { last PROXY_CHECK if $host !~ /localhost|127.0.0.1/; @@ -122,31 +121,32 @@ sub prepare_path { last PROXY_CHECK unless $ENV{HTTP_X_FORWARDED_HOST}; $host = $ENV{HTTP_X_FORWARDED_HOST}; - # backend could be on any port, so + + # backend could be on any port, so # assume frontend is on the default port $port = $c->request->secure ? 443 : 80; } my $path = $base_path . $ENV{PATH_INFO}; $path =~ s{^/+}{}; - + my $uri = URI->new; - $uri->scheme( $scheme ); - $uri->host( $host ); - $uri->port( $port ); - $uri->path( $path ); + $uri->scheme($scheme); + $uri->host($host); + $uri->port($port); + $uri->path($path); $uri->query( $ENV{QUERY_STRING} ) if $ENV{QUERY_STRING}; - + # sanitize the URI $uri = $uri->canonical; - $c->request->uri( $uri ); + $c->request->uri($uri); # set the base URI # base must end in a slash - $base_path .= '/' unless ( $base_path =~ /\/$/ ); + $base_path .= '/' unless ( $base_path =~ /\/$/ ); my $base = $uri->clone; - $base->path_query( $base_path ); - $c->request->base( $base ); + $base->path_query($base_path); + $c->request->base($base); } =item $self->prepare_query_parameters($c) @@ -156,8 +156,9 @@ sub prepare_path { sub prepare_query_parameters { my ( $self, $c ) = @_; - my $u = URI::Query->new( $ENV{QUERY_STRING} ); - $c->request->query_parameters( { $u->hash } ); + if ( $ENV{QUERY_STRING} ) { + $self->SUPER::prepare_query_parameters( $c, $ENV{QUERY_STRING} ); + } } =item $self->prepare_write($c) @@ -168,18 +169,18 @@ Enable autoflush on the output handle for CGI-based engines. sub prepare_write { my ( $self, $c ) = @_; - + # Set the output handle to autoflush - $c->response->handle->autoflush(1); - - $self->NEXT::prepare_write( $c ); + *STDOUT->autoflush(1); + + $self->NEXT::prepare_write($c); } =item $self->read_chunk($c, $buffer, $length) =cut -sub read_chunk { shift; shift->request->handle->sysread( @_ ); } +sub read_chunk { shift; shift; *STDIN->sysread(@_); } =item $self->run