X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FCatalyst%2FController%2FWrapCGI.pm;h=1a8447c0ca5382f18c2654dc7e4dc916ce36ce5d;hb=5464705b4645ee3f5c5694cbc8e1d9e8f12da983;hp=11f2fe76acae586d0cec72aa6786e8cc3e45d3ac;hpb=0d83c5de80494ceb3f8bf7c39206b477d5f314e5;p=catagits%2FCatalyst-Controller-WrapCGI.git diff --git a/lib/Catalyst/Controller/WrapCGI.pm b/lib/Catalyst/Controller/WrapCGI.pm index 11f2fe7..1a8447c 100644 --- a/lib/Catalyst/Controller/WrapCGI.pm +++ b/lib/Catalyst/Controller/WrapCGI.pm @@ -9,6 +9,10 @@ use HTTP::Request::AsCGI (); use HTTP::Request (); use URI (); use Catalyst::Exception (); +use URI::Escape; +use HTTP::Request::Common; + +use namespace::clean -except => 'meta'; =head1 NAME @@ -16,11 +20,11 @@ Catalyst::Controller::WrapCGI - Run CGIs in Catalyst =head1 VERSION -Version 0.0030 +Version 0.022 =cut -our $VERSION = '0.0030'; +our $VERSION = '0.022'; =head1 SYNOPSIS @@ -48,7 +52,7 @@ In your .conf, configure which environment variables to pass: pass_env PERL5LIB pass_env PATH pass_env /^MYAPP_/ - kill_env MOD_PERL + kill_env MYAPP_BAD @@ -62,22 +66,32 @@ with L 5.8 onward. If you just want to run CGIs from files, see L. +C will be set to C<< $c->user->obj->$username_field >> if +available, or to C<< $c->req->remote_user >> otherwise. + =head1 CONFIGURATION +=head2 pass_env + C<< $your_controller->{CGI}{pass_env} >> should be an array of environment variables or regular expressions to pass through to your CGIs. Entries surrounded by C characters are considered regular expressions. +=head2 kill_env + C<< $your_controller->{CGI}{kill_env} >> should be an array of environment variables or regular expressions to remove from the environment before passing it to your CGIs. Entries surrounded by C characters are considered regular expressions. -Default is to pass the whole of C<%ENV>, except for C (that is, the -default C is C<[ 'MOD_PERL' ]>. +Default is to pass the whole of C<%ENV>, except for entries listed in +L below. + +=head2 username_field -C<< $your_controller->{CGI}{username_field} >> should be the field for your user's name, which will be -read from C<< $c->user->obj >>. Defaults to 'username'. +C<< $your_controller->{CGI}{username_field} >> should be the field for your +user's name, which will be read from C<< $c->user->obj >>. Defaults to +'username'. See L for an example. @@ -91,12 +105,14 @@ open my $REAL_STDOUT, ">>&=".fileno(*STDOUT); =head1 METHODS -=head2 $self->cgi_to_response($c, $coderef) +=head2 cgi_to_response + +C<< $self->cgi_to_response($c, $coderef) >> Does the magic of running $coderef in a CGI environment, and populating the appropriate parts of your Catalyst context with the results. -Calls wrap_cgi (below.) +Calls L. =cut @@ -119,18 +135,21 @@ sub cgi_to_response { $c->res->headers($res->headers); } -=head2 $self->wrap_cgi($c, $coderef) +=head2 wrap_cgi + +C<< $self->wrap_cgi($c, $coderef) >> Runs $coderef in a CGI environment using L, returns an L. -The CGI environment is set up based on $c. +The CGI environment is set up based on C<$c>. The environment variables to pass on are taken from the configuration for your Controller, see L for an example. If you don't supply a list of -environment variables to pass, the whole of %ENV is used. +environment variables to pass, the whole of %ENV is used (with exceptions listed +in L. -Used by cgi_to_response (above), which is probably what you want to use as well. +Used by L, which is probably what you want to use as well. =cut @@ -148,7 +167,28 @@ sub wrap_cgi { local $/; $body_content = <$body>; } else { my $body_params = $c->req->body_parameters; - if (%$body_params) { + + if (my %uploads = %{ $c->req->uploads }) { + my $post = POST 'http://localhost/', + Content_Type => 'form-data', + Content => [ + %$body_params, + map { + my $upl = $uploads{$_}; + $_ => [ + undef, + $upl->filename, + Content => $upl->slurp, + map { + my $header = $_; + map { $header => $_ } $upl->headers->header($header) + } $upl->headers->header_field_names + ] + } keys %uploads + ]; + $body_content = $post->content; + $req->content_type($post->header('Content-Type')); + } elsif (%$body_params) { my $encoder = URI->new; $encoder->query_form(%$body_params); $body_content = $encoder->query; @@ -167,13 +207,20 @@ sub wrap_cgi { ? eval { $c->user->obj->$username_field } : ''); - my $path_info = '/'.join '/' => @{ $c->req->args }; + $username ||= $c->req->remote_user if $c->req->can('remote_user'); + + my $path_info = '/'.join '/' => map { + utf8::is_utf8($_) ? uri_escape_utf8($_) : uri_escape($_) + } @{ $c->req->args }; my $env = HTTP::Request::AsCGI->new( $req, ($username ? (REMOTE_USER => $username) : ()), %$filtered_env, - PATH_INFO => $path_info + PATH_INFO => $path_info, +# eww, this is likely broken: + FILEPATH_INFO => '/'.$c->action.$path_info, + SCRIPT_NAME => $c->uri_for($c->action, $c->req->captures)->path ); { @@ -199,6 +246,44 @@ sub wrap_cgi { return $env->response; } +=head1 FILTERED ENVIRONMENT + +If you don't use the L option to restrict which environment variables +are passed in, the default is to pass the whole of C<%ENV> except the variables +listed below. + + MOD_PERL + SERVER_SOFTWARE + SERVER_NAME + GATEWAY_INTERFACE + SERVER_PROTOCOL + SERVER_PORT + REQUEST_METHOD + PATH_INFO + PATH_TRANSLATED + SCRIPT_NAME + QUERY_STRING + REMOTE_HOST + REMOTE_ADDR + AUTH_TYPE + REMOTE_USER + REMOTE_IDENT + CONTENT_TYPE + CONTENT_LENGTH + HTTP_ACCEPT + HTTP_USER_AGENT + +C<%ENV> can be further trimmed using L. + +=cut + +my $DEFAULT_KILL_ENV = [qw/ + MOD_PERL SERVER_SOFTWARE SERVER_NAME GATEWAY_INTERFACE SERVER_PROTOCOL + SERVER_PORT REQUEST_METHOD PATH_INFO PATH_TRANSLATED SCRIPT_NAME QUERY_STRING + REMOTE_HOST REMOTE_ADDR AUTH_TYPE REMOTE_USER REMOTE_IDENT CONTENT_TYPE + CONTENT_LENGTH HTTP_ACCEPT HTTP_USER_AGENT +/]; + sub _filtered_env { my ($self, $env) = @_; my @ok; @@ -208,7 +293,7 @@ sub _filtered_env { $pass_env = [ $pass_env ] unless ref $pass_env; my $kill_env = $self->{CGI}{kill_env}; - $kill_env = [ 'MOD_PERL' ] unless defined $kill_env; + $kill_env = $DEFAULT_KILL_ENV unless defined $kill_env; $kill_env = [ $kill_env ] unless ref $kill_env; if (@$pass_env) { @@ -235,27 +320,36 @@ sub _filtered_env { return { map {; $_ => $env->{$_} } @ok }; } +__PACKAGE__->meta->make_immutable; -=head1 ACKNOWLEDGEMENTS +=head1 DIRECT SOCKET/NPH SCRIPTS -Original development sponsored by L +This currently won't work: -=head1 SEE ALSO + #!/usr/bin/perl -L, L, -L, L, L + use CGI ':standard'; -=head1 AUTHORS + $| = 1; -Originally written by: + print header; -Matt S. Trout, C<< >> + for (0..1000) { + print $_, br, "\n"; + sleep 1; + } -Contributors: +because the coderef is executed synchronously with C pointing to a temp +file. -Rafael Kitover C<< >> +=head1 ACKNOWLEDGEMENTS -Hans Dieter Pearcey C<< >> +Original development sponsored by L + +=head1 SEE ALSO + +L, L, +L, L, L =head1 BUGS @@ -289,9 +383,20 @@ L =back +=head1 AUTHOR + +Matt S. Trout C<< >> + +=head1 CONTRIBUTORS + +Rafael Kitover C<< >> + +Hans Dieter Pearcey C<< >> + =head1 COPYRIGHT & LICENSE -Copyright (c) 2008 Matt S. Trout +Copyright (c) 2008-2009 L and +L. This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself.