Re: Annoyance in the deployer

[Remy Maucherat]

Bill Barker wrote:
Hi,
Not something major, but there's that in the deployer:
               // Make sure there is an application configuration 
directory
               // This is needed if the Context appBase is the same 
as the
               // web server document root to make sure only web 
applications
               // are deployed and not directories for web space.
               File webInf = new File(dir, "/WEB-INF");
               if (!webInf.exists() || !webInf.isDirectory() ||
                   !webInf.canRead())
                   continue;

Is that really useful ? I ask because it prevents dropping a folder 
containing a bunch of files in webapps (without creating a stupid 
WEB-INF folder), which could be annoying.

Actually, I recently had a case where someone accidentally deleted 
WEB-INF on a development box.  When the Context didn't load, the 
problem was noticed immediately :).
Yes, I guess it's easier to spot that particular problem :)
Personally, I like the check (but won't veto the change).  But then, I 
believe that dropping in a folder containing a bunch of static files 
is what the Alias directive in Apache/httpd is for ;-).
Ok.
Maybe I can see if we could try to make recommendations for that kind of 
defaults and behaviors in the specification.

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