On Oct 9, 2007, at 5:33 AM, Ruediger Pluem wrote:
On 10/09/2007 01:47 AM, [EMAIL PROTECTED] 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?