Re: Resources - merging back the changes

2012-11-01 Thread Mark Thomas
On 01/11/2012 18:25, Mark Thomas wrote: > On 01/11/2012 05:21, Costin Manolache wrote: >> On Wed, Oct 31, 2012 at 4:41 PM, Costin Manolache wrote: >> >>> Hi Mark, >>> >>> I synced from HEAD - and noticed that DefaultServlet.PUT now fails with >>> '403' if the resource already exists. >>> I'm prett

Re: Resources - merging back the changes

2012-11-01 Thread Mark Thomas
On 01/11/2012 05:21, Costin Manolache wrote: > On Wed, Oct 31, 2012 at 4:41 PM, Costin Manolache wrote: > >> Hi Mark, >> >> I synced from HEAD - and noticed that DefaultServlet.PUT now fails with >> '403' if the resource already exists. >> I'm pretty sure this used to work, and at least for webda

Re: Resources - merging back the changes

2012-10-31 Thread Costin Manolache
On Wed, Oct 31, 2012 at 4:41 PM, Costin Manolache wrote: > Hi Mark, > > I synced from HEAD - and noticed that DefaultServlet.PUT now fails with > '403' if the resource already exists. > I'm pretty sure this used to work, and at least for webdav it's supposed > to replace the content. > Is this a

Re: Resources - merging back the changes

2012-10-31 Thread Costin Manolache
Hi Mark, I synced from HEAD - and noticed that DefaultServlet.PUT now fails with '403' if the resource already exists. I'm pretty sure this used to work, and at least for webdav it's supposed to replace the content. Is this a bug - or intended (the problem seems to be preResourceExists() check in

Re: Resources - merging back the changes

2012-10-23 Thread Mark Thomas
On 23/10/2012 01:01, Konstantin Kolinko wrote: > 2012/10/17 Mark Thomas : >> On 17/10/2012 15:07, Konstantin Kolinko wrote: >> >>> 1. Are and able to inject individual files? >> >> Not yet, but that should be relatively simple to implement. >> >>> 2. I am -1 on removal of VirtualWebappLoader. >>>

Re: Resources - merging back the changes

2012-10-22 Thread Konstantin Kolinko
2012/10/17 Mark Thomas : > On 17/10/2012 15:07, Konstantin Kolinko wrote: > >> 1. Are and able to inject individual files? > > Not yet, but that should be relatively simple to implement. > >> 2. I am -1 on removal of VirtualWebappLoader. >> >> The feature of injecting additional classpath entries

Re: Resources - merging back the changes

2012-10-17 Thread Mark Thomas
On 17/10/2012 15:07, Konstantin Kolinko wrote: > 1. Are and able to inject individual files? Not yet, but that should be relatively simple to implement. > 2. I am -1 on removal of VirtualWebappLoader. > > The feature of injecting additional classpath entries into > WebappClassLoader.super.add

Re: Resources - merging back the changes

2012-10-17 Thread Konstantin Kolinko
2012/10/16 Mark Thomas : > On 09/10/2012 22:27, Mark Thomas wrote: >> I believe from the various threads on the Resources implementation for >> Tomcat 8 there is agreement that: >> >> - DirContext is not the right basis for the Resources API >> - Refactoring is required to simplify the multiple 'ad

Re: Resources - merging back the changes

2012-10-16 Thread Mark Thomas
On 09/10/2012 22:27, Mark Thomas wrote: > I believe from the various threads on the Resources implementation for > Tomcat 8 there is agreement that: > > - DirContext is not the right basis for the Resources API > - Refactoring is required to simplify the multiple 'add-ons' and to > provide a sou

Resources - merging back the changes

2012-10-09 Thread Mark Thomas
I believe from the various threads on the Resources implementation for Tomcat 8 there is agreement that: - DirContext is not the right basis for the Resources API - Refactoring is required to simplify the multiple 'add-ons' and to provide a sound basis for overlays (which may slip to Servlet 3.2