You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by bu...@apache.org on 2010/05/01 14:13:08 UTC

DO NOT REPLY [Bug 49218] Fix support for META-INF/resources

https://issues.apache.org/bugzilla/show_bug.cgi?id=49218

--- Comment #5 from Konstantin Kolinko <kn...@gmail.com> 2010-05-01 08:13:02 EDT ---
(In reply to comment #4)
TestStandardContextResources calling resourceF.jsp in r939650 checks for wrong
value. It should look for "resourceF.jsp in resources.jar".

Besides that, it really uncovered a bug. The test fails because of the
following:

5. In BaseDirContext#lookup():
altDirContext.lookup("META-INF/resources/" + name) does not return null, but
throws a NamingException, which aborts the for( : altDirContexts) cycle.

So, either lookup for resourceF.jsp or for resourceF.jsp will fail, depending
on the order in altDirContexts (the spec says that the order is arbitrary).


6. Also in BaseDirContext#lookup():
doLookup(name) call is also documented to throw NamingException.
Either it does not throw it and the doLookup method signature has to be updated
(or at least add a JavaDoc there), or the code is broken.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org