Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix

[David Rees]

On Tue, Aug 14, 2001 at 11:49:43PM -0400, Keith Wannamaker wrote:
> Try ap_escape_uri

That does the trick.

Here's the patch which gets things working again, thanks for all the help. 
Hopefully this will get applied soon.  Is there any 3.2.4 release planned to
fix the small number of bugs/problems in 3.2.3 (I also recall bumping into
some issues with error documents and getting into infinite loops which were
fixed)

Thanks,
Dave

--- mod_jk.c.orig       Tue Jun 19 15:44:57 2001
+++ mod_jk.c    Tue Aug 14 22:42:32 2001
@@ -358,13 +358,12 @@
     s->method       = (char *)r->method;
     s->content_length = get_content_length(r);
     s->query_string = r->args;
-    s->req_uri      = r->unparsed_uri;
-    if (s->req_uri != NULL) {
-       char *query_str = strchr(s->req_uri, '?');
-       if (query_str != NULL) {
-           *query_str = 0;
-       }
-    }
+    /*
+     * The 2.2 servlet spec errata says the uri from
+     * HttpServletRequest.getRequestURI() should remain encoded.
+     * [http://java.sun.com/products/servlet/errata_042700.html]
+     */
+    s->req_uri = ap_escape_uri(r->pool, r->uri);
 
     s->is_ssl       = JK_FALSE;
     s->ssl_cert     = NULL;