Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c jk_uri_worker_map.h

2005-06-24 Thread Derrick Koes
Developers List Subject: Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c jk_uri_worker_map.h Date: Thu, 23 Jun 2005 09:43:37 +0200 Derrick Koes wrote: I tested yesterday's CVS head for compliance with session ID URL rewriting. This fails. The jsessionid

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c jk_uri_worker_map.h

2005-06-23 Thread Mladen Turk
Derrick Koes wrote: I tested yesterday's CVS head for compliance with session ID URL rewriting. This fails. The jsessionid is lost from the URL. Well, I tested that with the /servlets-examples/servlet/SessionExample and the jsessionid's *are* preserved. Can you post the 'JkLogLevel debug'

RE: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c jk_uri_worker_map.h

2005-06-22 Thread Derrick Koes
I tested yesterday's CVS head for compliance with session ID URL rewriting. This fails. The jsessionid is lost from the URL. Derrick From: [EMAIL PROTECTED] Reply-To: "Tomcat Developers List" To: [EMAIL PROTECTED] Subject: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_work

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Mladen Turk
Bill Barker wrote: and requesting: http://myserver//myapp/index.jsp Yes I did, and it really doesn't mater. Well my jsp's are inside tomcat application, so I admit there can be problems if the .jsp is hanging around somewhere outside web application :). It's for when you want Apache to act as the

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Bill Barker
- Original Message - From: "Mladen Turk" <[EMAIL PROTECTED]> To: "Tomcat Developers List" <[EMAIL PROTECTED]> Sent: Friday, December 17, 2004 12:47 PM Subject: Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c > Bill

Re: another data point RE: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Mladen Turk
Derrick Koes wrote: The fix made early this morning (~7-8am Eastern) works to fix the problem I was having with URL rewriting. I'll grab the latest code set again soon and retry. Cool, those are really good news! Glad that it works finally :). Regards, Mladen. -

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Mladen Turk
Bill Barker wrote: Try with only: JkMount /myapp/*.jsp and requesting: http://myserver//myapp/index.jsp Yes I did, and it really doesn't mater. Well my jsp's are inside tomcat application, so I admit there can be problems if the .jsp is hanging around somewhere outside web application :). Also I've

another data point RE: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Derrick Koes
rence I've found to this point. Derrick -Original Message- From: Mladen Turk [mailto:[EMAIL PROTECTED] Sent: Friday, December 17, 2004 2:59 PM To: Tomcat Developers List Subject: Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c Bill Barker wrote: &

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Bill Barker
- Original Message - From: "Mladen Turk" <[EMAIL PROTECTED]> To: "Tomcat Developers List" <[EMAIL PROTECTED]> Sent: Friday, December 17, 2004 11:59 AM Subject: Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c > Bill Bark

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Mladen Turk
Bill Barker wrote: But since the request is supposed to be atomic why to strdup an uri? I'd rather remove char *uri = apr_pstrdup(r->pool, r->uri); before calling map_uri_to_worker then adding strdup to IIS. It was done to fix a '//' bypass traversal bug (e.g. http://myserver/myapp//foo.jsp would s

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Bill Barker
- Original Message - From: "Mladen Turk" <[EMAIL PROTECTED]> To: "Tomcat Developers List" <[EMAIL PROTECTED]> Sent: Friday, December 17, 2004 9:34 AM Subject: Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c > Bill B

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Mladen Turk
Bill Barker wrote: No, it's not related to IIS at all. Of course it is related to IIS, since IIS is the only one that (wrongly) doesn't pass a copy of the URI to map_uri_to_worker OK. Seems that you catch me on that :). But since the request is supposed to be atomic why to strdup an uri? I'd rath

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Bill Barker
- Original Message - From: "Mladen Turk" <[EMAIL PROTECTED]> To: "Tomcat Developers List" <[EMAIL PROTECTED]> Sent: Friday, December 17, 2004 8:06 AM Subject: Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c Bill Ba

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Mladen Turk
Bill Barker wrote: I'm -1 on this patch. map_uri_to_worker doesn't need all of this spaghetti just because IIS is broken. Also you can see the difference with Apache2. Open http://host/servlets-examples/servlet/SessionExample Now, click on the 'Url encoded' link. You will see the ';jsessionid=.

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Mladen Turk
Bill Barker wrote: I'm -1 on this patch. map_uri_to_worker doesn't need all of this spaghetti just because IIS is broken. Continuing from last reply... Also, I'm stunned with the fact that there is no reports in Bugzilla about that. I mean we have used the uri: /uri;jsessionid=?params and ma

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Mladen Turk
Bill Barker wrote: I'm -1 on this patch. map_uri_to_worker doesn't need all of this spaghetti just because IIS is broken. No, it's not related to IIS at all. The previous implementation was taking the uri an cut out the ';jseessionid=...' from the url. Take a look at JK2 and it's jk_uriMap.c: ur

Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c

2004-12-17 Thread Bill Barker
- Original Message - From: <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Friday, December 17, 2004 3:24 AM Subject: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c mturk 2004/12/17 03:24:38 Modified:jk/native/common jk_uri_worker_map.c Log: Fi