On Tue, Aug 14, 2001 at 11:05:38PM -0400, Keith Wannamaker wrote: > | This breaks query strings. > | > | r->uri contains only the path portion of the URL. r->unparsed_uri > | contains the URL in its "virgin" format - as sent by the client. > > No, I don't believe this is quite right. > getRequestURI() in a servlet should return > r->unparsed_uri minus a query string. > > Setting s->uri = r->uri doesn't break > query strings.. but it *does* break the > encoding of the uri. > > So tc 3.3 is currently broken as is mod_webapp > (unless the string is encoded on the java side > in TC4). > > However, Justin, I think your suggestion is > the correct solution: > > s->req_uri = ap_encode_uri(r->pool, r->uri); > > David, or anyone else interested too, would you > try this with some corner test cases and see if > it lives up to expectation? I gave it a shot and it compiled fine, but got this error at runtime: Cannot load /usr/local/apache/libexec/mod_jk.so into server: /usr/local/apache/libexec/mod_jk.so: undefined symbol: ap_encode_uri Any hints? I'm new at Apache module hacking. -Dave
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Justin Erenkrantz
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Pier P. Fumagalli
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Craig R. McClanahan
- RE: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Keith Wannamaker
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix cmanolache
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Justin Erenkrantz
- RE: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Keith Wannamaker
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix cmanolache
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Justin Erenkrantz
- RE: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix David Rees
- RE: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Keith Wannamaker
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix David Rees
- RE: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Keith Wannamaker
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Justin Erenkrantz
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix cmanolache
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Bill Barker
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix cmanolache
- Fw: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix Bill Barker
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix David Rees
- Re: [TC3.2.3][PATCH] mod_jk / mod_rewrite bug fix cmanolache