Hi Dave,
As I understand from the jk2 source code, the main apache server (that is not a
virtual host)
is reffered to in mod_jk2 code as NULL (for uri mapping purposes).
Following this logic the check is right.
I did not have this check in my fix because all my
hosts in apache httpd.conf are virtual.
I have not tested anything yet - it is a weekend after all
Dmitry
From: Short, Dave [mailto:[EMAIL PROTECTED]]
Sent: Сб 21/09/2002 1:01
To: 'Tomcat Users List'
CC:
Subject: RE: mod_jk2, virtual hosts, JkUriSet
Dmitry,
It looks like the code the developers added was different from your original
code.
Here's your code:
uriEnv = workerEnv->uriMap->mapUri(env, workerEnv->uriMap,NULL,r->uri);
"NULL" should be replaced to "r->server->server_hostname" (without the
quotes of course)
and here's their code:
- uriEnv = workerEnv->uriMap->mapUri(env, workerEnv->uriMap,NULL,r->uri
);
+ uriEnv = workerEnv->uriMap->mapUri(env, workerEnv->uriMap,
+ r->server->is_virtual ? r->server->server_hostname :
NULL,
+ r->uri );
Looks like they added a check for a virtual host before executing your code.
Have you had a chance to test this?
-----Original Message-----
From: Dmitry Letin [mailto:[EMAIL PROTECTED]]
Sent: September 21, 2002 8:50 AM
To: [EMAIL PROTECTED]
Subject: Re: mod_jk2, virtual hosts, JkUriSet
Developers have already added this fix:
http://www.mail-archive.com/tomcat-dev%40jakarta.apache.org/msg33556.html
That was really fast.
Dmitry
----- Original Message -----
From: Robert L Sowders [mailto:[EMAIL PROTECTED]]
Sent: Сб 21/09/2002 2:16
To: Tomcat Users List
CC: Dmitry Letin
Subject: Re: mod_jk2, virtual hosts, JkUriSet
Hello,
I'm glad you got it working. Have you posted a bug at
http://www.mail-archive.com/tomcat-dev%40jakarta.apache.org/ I'm
sure the
developers of JK2 would be interested in your patch?
You might have already, their web site has not been updating for
awhile
now.
rls
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
<<winmail.dat>>
-- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
