httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ruediger Pluem <rpl...@apache.org>
Subject Re: svn commit: r583002 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy_http.c
Date Tue, 09 Oct 2007 09:33:07 GMT


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.

Regards

RĂ¼diger

>

Mime
View raw message