httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: svn commit: r583002 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c
Date Tue, 09 Oct 2007 13:03:59 GMT

On Oct 9, 2007, at 5:33 AM, Ruediger Pluem wrote:

>
>
> On 10/09/2007 01:47 AM, niq@apache.org wrote:
>> Author: niq
>> Date: Mon Oct  8 16:47:35 2007
>> New Revision: 583002
>>
>> URL: http://svn.apache.org/viewvc?rev=583002&view=rev
>> Log:
>> mod_proxy_http: Don't unescape/escape forward proxied URLs.  Just  
>> check them.
>> PR 42592
>>
>> also add fix to PR42572 to CHANGES (from r563487/r563489)
>>
>> Modified:
>>     httpd/httpd/trunk/CHANGES
>>     httpd/httpd/trunk/modules/proxy/mod_proxy_http.c
>>
>
>> Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_http.c
>> URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/ 
>> mod_proxy_http.c?rev=583002&r1=583001&r2=583002&view=diff
>> ===================================================================== 
>> =========
>> --- httpd/httpd/trunk/modules/proxy/mod_proxy_http.c (original)
>> +++ httpd/httpd/trunk/modules/proxy/mod_proxy_http.c Mon Oct  8  
>> 16:47:35 2007
>> @@ -36,6 +36,9 @@
>>      const char *err;
>>      const char *scheme;
>>      apr_port_t port, def_port;
>> +    const char *p;
>> +    const char *allowed = "~$-_.+!*'(),;:@&=/"; /* allowed 
>> +reserved from
>> +                                                    
>> ap_proxy_canonenc */
>
> *allowed could be made 'static'.
>
> Otherwise looks good.
>

Would almost make sense to have this as an API function... anyone have
issues if I make the required adjustments for that to happen?


Mime
View raw message