Adding SERVER_PORT to bootstrap.cgi's

[Michael_E_Brown at Dell.com]

I've dropped this modification into the mirror cgi for the software/testing repositories. It should mirror externally within an hour. Please test and see that it operates as expected.
--
Michael


-----Original Message-----
From: Flaherty, Patrick [mailto:pflaherty at wsi.com]
Sent: Tue 3/18/2008 9:51 PM
To: Brown, Michael E; firmware-tools-devel at lists.us.dell.com
Subject: RE: Adding SERVER_PORT to bootstrap.cgi's
 
> No objection. Is SERVER_PORT a standard CGI variable?
> 
As far as I can tell it is. I've used it in perl cgi's on apache before,
and it worked on my bootstrap.cgi (well until rsync blew away the
changes)

Here's a link talking about default cgi vars.
http://hoohoo.ncsa.uiuc.edu/cgi/env.html


> 
> -----Original Message-----
> From: firmware-tools-devel-bounces at lists.us.dell.com on 
> behalf of Flaherty, Patrick
> Sent: Tue 3/18/2008 5:17 PM
> To: firmware-tools-devel at lists.us.dell.com
> Subject: Adding SERVER_PORT to bootstrap.cgi's
>  
> Would there be any opposition to adding SERVER_PORT to the 
> bootstrap.cgi files? Is there a place to file a feature request?
> 
> Example:
>  my $server_name = $ENV{"SERVER_NAME"} || "linux.dell.com";
> 
> Might become:
>  my $server_name = $ENV{"SERVER_NAME"}  || "linux.dell.com";  
> if ($ENV{"SERVER_PORT"} != 80 ) {
>   $server_name .= ":" . $ENV{"SERVER_PORT"};  }
> 
> Best
> Patrick
> 
> _______________________________________________
> firmware-tools-devel mailing list
> firmware-tools-devel at lists.us.dell.com
> http://lists.us.dell.com/mailman/listinfo/firmware-tools-devel
> 
> 
>