Mark Thomas wrote:

Resending. I have been having email problems recently. Apologies if this is received twice.

The attached patches address a couple of issue highlighted by bug 23267
- zero length contexts caused a NPE on contextFile.getParentFile()
- another exception on contextPath.substring(1) for zero length contexts

I also removed the filename.equals("ROOT") test as I couldn't see anywhere in the docs where this is specified as mandatory.

Patches for TC5 and TC4 both attached (untested on TC5)

I can confirm I think the patch is bad. For TC 4, this shouldn't be fixed (or likely all the changes to the deployer should also be ported). For TC 5, the problem was that the docbase was incorrectly set when unpacking a WAR in the case of the ROOT webapp.


For the deployer to do everything automatically, the resources names must match: context file, context path, exploded dir, WAR name.

Remy



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to