tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Remy Maucherat <r...@apache.org>
Subject Re: Annoyance in the deployer
Date Sun, 05 Dec 2004 16:56:54 GMT
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: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org


Mime
View raw message