DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=34994>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=34994


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO




------- Additional Comments From [EMAIL PROTECTED]  2005-05-27 06:05 -------
i took a look at your patch, very nice work!

there are a couple issues with it:

* it uses the authenticator action which does no longer exist:

+<map:match pattern="**">
+  <map:match type="agent-match" pattern="*DAV*">
+    <map:act type="authenticator">
+      <map:act type="authorizer">
+        <map:mount uri-prefix="" src="global-sitemap.xmap" check-reload="true"
reload-method="synchron"/> 
+      </map:act>
+      <map:act type="set-header">
+        <map:parameter name="WWW-Authenticate" value="Basic Realm=lenya" />
+        <map:generate type="html" src="401error.html"/>
+        <map:serialize type="xhtml" status-code="401"/>
+      </map:act>
+    </map:act>
+    <map:act type="set-header">
+      <map:parameter name="WWW-Authenticate" value="Basic Realm=lenya" />
+      <map:generate type="html" src="401error.html"/>
+      <map:serialize type="xhtml" status-code="401"/>
+    </map:act>
+  </map:match>
+</map:match>

an alternative will have to be found. the same can be said for the dicover
checkout action: actions are deprecated and should not be used for new 
development.

i also rewrote your patch to be more self-contained (putting all files under
davmap into default/lenya/usecases/webdav/) and also renaming
davmap/sitemap.xmap to webdav.xmap

the dependency chain for required cocoon blocks is:

webdav -> repository -> eventcache -> jms

also, it was a lot of work to integrate. please provide a unified diff in the
future. you can do this by right-clicking in eclipse and team-> create patch.

this is definitely something we want asap, i hope the devs chime in with
improvements. i have attached a unified diff with my changes.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

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

Reply via email to