Jan Luehe wrote:
>
> Remy Maucherat wrote:
>>I hadn't noticed you were the one who filed the bug. Besides skipping
>>the result.append(":");
>
> Not sure I understand: the ":" following the protocol is necessary,
> so that the printable representation of the context generated URLs
> starts with "jndi:".
Right, it's not related to the port number. Doh.
>>you should simply apply the fix, it's a very
>>good solution.
>
>
> I'm still having problems with CVS, which are preventing me from
> committing my fix.
>
> If you can commit this on my behalf, I'd appreciate it.
> The complete fix consists of the above diff and the following patch
> in org.apache.catalina.core.ApplicationContext:
>
> try {
> resources.lookup(path);
> return new URL
> - ("jndi", null, 0, getJNDIUri(hostName, fullPath),
> - new DirContextURLStreamHandler(resources));
> + ("jndi", "", 0, getJNDIUri(hostName, fullPath),
> + new DirContextURLStreamHandler(resources));
> } catch (Exception e) {
> // Ignore
> }
Ok.
Rémy
---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org
|