Hi Devs, Sergiu,
- I wasn't able to create new wiki documents. Is there something special I should do? Is it not supported yet? Is Konqueror alone guilty? - Also, the error I get doesn't seem right (File already exists), since the file I want to create definitely doesn't exist.
Adding pages / spaces is supported and unlike with some other features (like deleting documents) creating documents work pretty ok with most of the clients I tested. You should file a JIRA issue for this, I will test it asap. But I need to install KDE first ;)
- Since it uses Basic authentication, we should advertise that authentication is not secure, thus it should only be used inside trusted intranets, over VLANs or over SSL.
I think this information should go into http://dev.xwiki.org/xwiki/bin/view/Drafts/XWikiWebDAV which is totally out-dated now, i'll start working on it.
Code related: - Looking a bit over the code, I noticed that we're using only a bit from the jackrabit-jcr-server library, and I was wondering if it would be possible to refactor all the code that needs it to use something else, so that we have one dependency in minus. - Also, I noticed that you're using slf4j as the logging framework, which doesn't fit with the rest of the XWiki code, since we're using commons-logging. Still, this is also the framework used by jackrabbit, so we can't get rid of it completely. Asiri, could you update the code to use commons-logging? - Although there are integration tests, some unit tests would be great. I know it's hard for the moment to do proper unit testing, since most of the code works closely with the old core, which doesn't contribute to a good testing scenario.
Will look into these. Also, I'm digging up a patch which will integrate webdav service into xwiki ui as in http://91.121.237.216/xwiki/bin/view/Main/.
Congratulations for the good work, Asiri!
Thanks. Btw, is it possible for me to get commit rights for the webdav module ? It's bit painful to go with patches; not for me but for the person i continuously bug... :)
And I forgot one more thing: - The code is currently in com.xpn.xwiki.plugin.webdav, which I don't quite agree with. The plugin package is for plugins, and it is somewhat deprecated (since we're planning to switch completely to components and remove the notion of 'plugins' per se). Thus, I'd like to rename it to com.xpn.xwiki.webdav for the moment (except the actual webdav plugin, which is rightly placed in the plugin package).
+1 - Asiri
-- Sergiu Dumitriu http://purl.org/net/sergiu/ _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs