Make @removeslash a no-op when applied to a request for '/'.

bdarnell · bdarnell · commit 88833c118f6a · 2011-02-11T20:25:06.000-08:00
This prevents a redirect loop as browsers interpret an empty redirect
as a redirect to /.
diff --git a/tornado/web.py b/tornado/web.py
@@ -954,10 +954,12 @@ def wrapper(self, *args, **kwargs):
         if self.request.path.endswith("/"):
             if self.request.method in ("GET", "HEAD"):
                 uri = self.request.path.rstrip("/")
-                if self.request.query: uri += "?" + self.request.query
-                self.redirect(uri)
-                return
-            raise HTTPError(404)
+                if uri:  # don't try to redirect '/' to ''
+                    if self.request.query: uri += "?" + self.request.query
+                    self.redirect(uri)
+                    return
+            else:
+                raise HTTPError(404)
         return method(self, *args, **kwargs)
     return wrapper
 
