dav.resource.py: RedirectResponse takes a full href, not a uri

[karpet]

--- resource.py (revision 17515)
+++ resource.py (working copy)
@@ -469,7 +469,7 @@
         # If this is a collection and the URI doesn't end in "/", redirect.
         #
         if self.isCollection() and request.uri[-1:] != "/":
-            return RedirectResponse(request.uri + "/")
+            return RedirectResponse(request.unparseURL(path=request.uri + "/"))

         def setHeaders(response):
             response = IResponse(response)

[rfmohr@…]

Is there any chance that this fix will be merged in soon? The current implementation violates the spec, and the patch above easily fixes the problem.

(I just spent a couple of hours tracking down some weird behavior, and I finally isolated it to this problem. The behavior is only triggered in certain fringe cases, but fixing this bug might save others some time in the future.)

[exarkun]

Supplying a unit test which fails without this fix will speed up its acceptance.

Otherwise, this'll get applied when someone else finds the time to figure out how to trigger the bad behavior and then writes a unit test for it and verifies this patch fixes the problem.