[Feature] revise webdav configuration mechanism#5993
[Feature] revise webdav configuration mechanism#5993dizzzz wants to merge 5 commits intoeXist-db:developfrom
Conversation
|
A question, asked out of pure curiosity, total ignorance, and everlasting gratitude for your work on WebDAV in eXist: is there a reason that WebDAV configuration is stored in webdav.properties and not in conf.xml? |
|
@joewiz that is a very valid question :-) I never thought about it, I think I added this file as a 'easy fix' a long time ago. That said, I am not sure if extensions should be configured from conf.xml, shouldn't it be the core only? |
|
I added a mechanism to conf.xml years ago that allows you to specify a config for each module, maybe @dizzzz that could help you to move your config into there within the WebDAV section? |
|
thnx @adamretter I know about the config section. |
|
@dizzzz Ah! Interesting to consider how a jar-only .xar package could have an editable configuration. Perhaps it could (a) ship with a default configuration that would be instantiated without the presence of a config file, and (b) it could check for the presence of a config file on disk (or in |
Make configuration code of the webDAV extension more simple and consistent.
As a bonus the jdom 1.x dependency is removed for 'production'. For tests it is is still needed.
Code has been improved mechanically a bit.