Re: getContext() - spec interpretation

2005-11-30 Thread Mark Thomas
Jan, This looks good to me. I have patches ready to go for TC4 and TC5. They are based on the existing code but implement the spirit of Bill's proposed solution and add handling for crossContext. I'll give people a little longer in case they want to add to this discussion and then I'll

Re: getContext() - spec interpretation

2005-11-29 Thread Mark Thomas
Jan, Jan Luehe wrote: Hi Mark, snip I agree the current javadocs of ServletContext.getContext() are ambiguous, and I agree we need to present this case to the Servlet EG. Maybe there is a small chance to have this resolved in the Servlet 2.5 timeframe ... I think it would be more useful to

Re: getContext() - spec interpretation

2005-11-28 Thread Cédrik LIME
At 18:20 27/11/2005 +, you wrote: Bill Barker wrote: I can't really dispute the interpretation of the spec, but the change is likely to break many more webapps then it fixes :(. I thought this too. However, I think we should implement the spec as is. If we start relaxing it here, then

Re: getContext() - spec interpretation

2005-11-28 Thread Jan Luehe
Hi Mark, Mark Thomas wrote On 11/24/05 11:34,: All, I have been looking at bug 13040 and reviewing the current getContext() implementation. I saw Remy's comment from some time ago when fixing some related bugs (http://marc.theaimsgroup.com/?l=tomcat-devm=106008981803343w=2) that this

Re: getContext() - spec interpretation

2005-11-27 Thread Mark Thomas
Bill Barker wrote: I can't really dispute the interpretation of the spec, but the change is likely to break many more webapps then it fixes :(. I thought this too. However, I think we should implement the spec as is. If we start relaxing it here, then where else do we relax it? If other

Re: getContext() - spec interpretation

2005-11-27 Thread Mark Thomas
Mark Thomas wrote: The complications I referred to in the SSI commit are: 1. The definition of uripath says it is a context path which for the root context would be . In the method description it says uripath must always start with /. This is not consistent. 2. The definition of uripath