[CalendarServer-changes] [2704] CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702

[source_changes at macosforge.org]

Revision: 2704
          http://trac.macosforge.org/projects/calendarserver/changeset/2704
Author:   cdaboo at apple.com
Date:     2008-07-16 10:33:20 -0700 (Wed, 16 Jul 2008)
Log Message:
-----------
Fix redirect behavior on collections to allow for query parameters.

Modified Paths:
--------------
    CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/lib-patches/Twisted/twisted.web2.dav.resource.patch
    CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/lib-patches/Twisted/twisted.web2.static.patch
    CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/twistedcaldav/extensions.py
    CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/twistedcaldav/resource.py

Modified: CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/lib-patches/Twisted/twisted.web2.dav.resource.patch
===================================================================
--- CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/lib-patches/Twisted/twisted.web2.dav.resource.patch	2008-07-16 14:52:28 UTC (rev 2703)
+++ CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/lib-patches/Twisted/twisted.web2.dav.resource.patch	2008-07-16 17:33:20 UTC (rev 2704)
@@ -895,7 +895,7 @@
      # HTTP
      ##
  
-@@ -1525,10 +1881,6 @@
+@@ -1525,15 +1881,11 @@
          #litmus = request.headers.getRawHeaders("x-litmus")
          #if litmus: log.msg("*** Litmus test: %s ***" % (litmus,))
  
@@ -906,6 +906,13 @@
          #
          # If this is a collection and the URI doesn't end in "/", redirect.
          #
+-        if self.isCollection() and request.uri[-1:] != "/":
+-            return RedirectResponse(request.uri + "/")
++        if self.isCollection() and request.path[-1:] != "/":
++            return RedirectResponse(request.unparseURL(path=request.path+'/'))
+ 
+         def setHeaders(response):
+             response = IResponse(response)
 @@ -1567,7 +1919,7 @@
      def findChildren(self, depth, request, callback, privileges=None, inherited_aces=None):
          return succeed(None)

Modified: CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/lib-patches/Twisted/twisted.web2.static.patch
===================================================================
--- CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/lib-patches/Twisted/twisted.web2.static.patch	2008-07-16 14:52:28 UTC (rev 2703)
+++ CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/lib-patches/Twisted/twisted.web2.static.patch	2008-07-16 17:33:20 UTC (rev 2704)
@@ -14,3 +14,12 @@
          # Remove the dots from the path to split
          self.defaultType = defaultType
          self.ignoredExts = list(ignoredExts)
+@@ -383,7 +386,7 @@
+             return responsecode.NOT_FOUND
+ 
+         if self.fp.isdir():
+-            if req.uri[-1] != "/":
++            if req.path[-1] != "/":
+                 # Redirect to include trailing '/' in URI
+                 return http.RedirectResponse(req.unparseURL(path=req.path+'/'))
+             else:

Modified: CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/twistedcaldav/extensions.py
===================================================================
--- CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/twistedcaldav/extensions.py	2008-07-16 14:52:28 UTC (rev 2703)
+++ CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/twistedcaldav/extensions.py	2008-07-16 17:33:20 UTC (rev 2704)
@@ -472,7 +472,7 @@
             return responsecode.NOT_FOUND
 
         if self.fp.isdir():
-            if request.uri[-1] != "/":
+            if request.path[-1] != "/":
                 # Redirect to include trailing '/' in URI
                 return RedirectResponse(request.unparseURL(path=request.path+'/'))
             else:

Modified: CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/twistedcaldav/resource.py
===================================================================
--- CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/twistedcaldav/resource.py	2008-07-16 14:52:28 UTC (rev 2703)
+++ CalendarServer/branches/users/cdaboo/trigger-auto-accept-2702/twistedcaldav/resource.py	2008-07-16 17:33:20 UTC (rev 2704)
@@ -117,7 +117,7 @@
 
         if not html_agent and self.isPseudoCalendarCollection():
             # Render a monolithic iCalendar file
-            if request.uri[-1] != "/":
+            if request.path[-1] != "/":
                 # Redirect to include trailing '/' in URI
                 return RedirectResponse(request.unparseURL(path=request.path+"/"))
 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.macosforge.org/pipermail/calendarserver-changes/attachments/20080716/ce3f9586/attachment.html