> hi again,
>
> this patch allows the user to disable the generation of html directory
> listings for requests on collection resources (as init-parameter).
>
> in addition I've moved the html-generating code for directory listing
> into a (new) subclass of GetMethod, with the (somewhat lengthy) name
> DirectoryBrowsingGetMethod. this class will do the directory display
> job for any collection resource request, but delegate to the GetMethod
> for 'normal' resources. the GetMethod class will return a 403 status
> for requests on collections.
>
> the rationale behind this is to separate the html-generating code from
> the normal GET handling, as
> (a) I think "real" sites won't use the default directory browsing
> anyway
I'm neutral on the subject, because :
- PROPFIND will return even richer information that the dir browse page
- I wouldn't see the Slide WebDAV servlet being accessible on a public
website in the forseeable future, mainly because it's extremely feature rich
(therefore, more vulnerable to exploits than a servlet which would only
serve GET requests)
I'll apply the patch, since it may be a functionality other people would
want.
> (b) that stuff should go into a slide jsp tag-library (for example) in
> the long term
> based on the 'directory-browsing' boolean init-parameter, the servlet
> will choose either the regular GetMethod or the
> DirectoryBrowsingGetMethod.
>
> [btw: is there a way to support welcome-files in the WebdavServlet ?]
You can always add a dead property on a collection, but I like the multiple
view concept better overall. There's an example with the Catalina
integration I did :
- View 1 : Standard web application
- View 2 : Edit the web application
- View 3 : Manage the web application
Each view is a separate context (located in a different host in Catalina),
so the view 2 allows to edit any resource in the webapp, including class
repositories and JSPs. If you use Slide's WebDAV servlet to replace
Catalina's default servlet, you'll end up with mapping confilcts (for
example, with Jasper) which would prevent using WebDAV on some resources.
Remy
