"as a second afterthought, my ($cgi, $CMD) = shift; looks strange. i would have used my ($cgi, $CMD) = @_; instead" Re^2: replace use HTTP::Lite to run a perl file that was on another server
Result:#!/usr/bin/perl -w my $all=get_response('a','b'); sub get_response { my ($cgi, $CMD) = shift; print 'a'.$cgi."\n"; print 'b'.$CMD."\n"; }
aa Use of uninitialized value $CMD in concatenation (.) or string at x118 +2315.pl line 6. b
is now in the same cgi-bin directorty as the perl script that is calling for that script. replace use HTTP::Lite to run a perl file that was on another server Taking that to mean that the caller was cgi as well i added as an afterthought, that &get_response clobbers %ENV, so you might want to add local %ENV=%ENV; as the first line in sub get_response Re^2: replace use HTTP::Lite to run a perl file that was on another server because i didnt want a clobbered %ENV to mess up some other part of the cgi script.
so probably best is
Tho in this case the 3 arg open didnt mean as much since we know that the $cgi parm had no special characters being hardcoded.sub get_response { local %ENV=%ENV; my ($cgi, $query) = @_; ## prepare proper (CGI) environment $ENV{QUERY_STRING} = $query; $ENV{REQUEST_METHOD} = 'GET'; $ENV{GATEWAY_INTERFACE} = 'CGI/1.1'; # add more as needed open(CGI, '-|', $cgi) or die "Can't exec $cgi, $!"; local $/ = undef; my $res = <CGI>; close(CGI) or die "Error running $cgi, $!"; return $res; }
well at least my first post had added Using get_response from that node and untested and i did look over the code again and again to make sure it was doing what i thought it was and posted addendums to correct what i thought were weaknesses.
In reply to Re^5: replace use HTTP::Lite to run a perl file that was on another server
by huck
in thread replace use HTTP::Lite to run a perl file that was on another server
by bNathan
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |