perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Philippe M. Chiasson" <>
Subject Re: [PATCH] Apache2::RequestUtil->add_config
Date Wed, 18 Jan 2006 01:41:12 GMT
Philippe M. Chiasson wrote:
> Torsten Foertsch wrote:
>>the patch below solves the following situation.
>>The ProxyPassReverse directive is outside any <Location> block given as
>>  ProxyPassReverse /path http://...
>>but inside a Location block without the /path specification:
>>  <Location /path>
>>  ProxyPassReverse http://...
>>  </Location>
>>In the latter case the path is taken from $parms->path, that means the path of

>>the Location block.
> [...]
>>The patch adds an optional 3rd parameter to $r->add_config that lets you pass 
>>in the path.
>>I don't think ProxyPassReverse is the only directive that behaves this way.
> As a more generic solution, this might make sense. On first glance, your patch
> looks sane, but I won't have the time to really look at it until Jan 15th

I've finally got around looking at this patch, and it looks good. I tweaked it
some (whitespace and formatting) and checked it in as r370001 & r370002.

Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

View raw message