Changeset f7cd67 in indico

Timestamp:

04/05/11 11:46:10 (2 years ago)

Author:

Pedro Ferreira <jose.pedro.ferreira@…>

Branches:

master, burotel, hello-world-walkthrough, ipv6, v0.98-series, v0.98.2, v0.98.3, v0.98b1, v0.98b2, v0.99, 051b2622c51afb171a1dedb46a0df4fbb0cbd02e, 0da0c1403bae8e51d8229f460181c71b9e6dda72

Children:

cf4381

Parents:

21f3fc

Message:

[FIX] short urls now accept trailing slashes

• Not best solution, though - regexp should have probably been used;
• We need to sort out this WSGIRedirect thing, as it does no actual redirect;

File:

• indico/web/wsgi/indico_wsgi_url_parser.py (modified) (2 diffs)

indico/web/wsgi/indico_wsgi_url_parser.py

@@ -100 +100 @@
 class WSGIRedirection(object):
     """
-    Redirections that need a especial treatment are handled through this class.
+    Redirections that need special treatment are handled by this class.
     """
     def __init__(self, req, module, handler, dir):
@@ -129 +129 @@
         """
         newQueryString = query + '='
-        newQueryString += '/'.join(self._path[self._index+1:])
+
+        # remove trailing slashes
+        filteredPath = self._path[self._index + 1:]
+        if filteredPath[-1] == '':
+            filteredPath = filteredPath[:-1]
+
+        newQueryString += '/'.join(filteredPath)
+
         if self._req.URLFields['QUERY_STRING']:
             newQueryString += '?' + self._req.URLFields['QUERY_STRING']