tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Barker" <>
Subject Re: cvs commit: jakarta-tomcat-connectors/jk/native/common jk_uri_worker_map.c
Date Fri, 17 Dec 2004 16:30:07 GMT

----- Original Message ----- 
From: "Mladen Turk" <>
To: "Tomcat Developers List" <>
Sent: Friday, December 17, 2004 8:06 AM
Subject: Re: cvs commit: jakarta-tomcat-connectors/jk/native/common 

> 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.

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

> 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:
> url is rewritten, but restored on exit that jk did not do.
> That is exactly what this patch does.

Yeah, well, JK2 is hardly a shining example of how things should be done 
;-).  jk_uriMap.c is well known to be seriously broken.

> Regards,
> Mladen
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

This message is intended only for the use of the person(s) listed above as the intended recipient(s),
and may contain information that is PRIVILEGED and CONFIDENTIAL.  If you are not an intended
recipient, you may not read, copy, or distribute this message or any attachment. If you received
this communication in error, please notify us immediately by e-mail and then delete all copies
of this message and any attachments.

In addition you should be aware that ordinary (unencrypted) e-mail sent through the Internet
is not secure. Do not send confidential or sensitive information, such as social security
numbers, account numbers, personal identification numbers and passwords, to us via ordinary
(unencrypted) e-mail.

View raw message