Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Extended @removeslash also to CRUD #70 #81

Merged
merged 2 commits into from

2 participants

@dhilipsiva

This closes issue #70

@fiorix fiorix merged commit fff4fa5 into fiorix:master
@fiorix
Owner

thanks!

@dhilipsiva

And thank you for giving us cyclone. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 5, 2012
  1. @dhilipsiva
  2. @dhilipsiva

    Update README.rst

    dhilipsiva authored
This page is out of date. Refresh to see the latest.
Showing with 5 additions and 1 deletion.
  1. +4 −0 README.rst
  2. +1 −1  cyclone/web.py
View
4 README.rst
@@ -688,3 +688,7 @@ Thanks to (in no particular order):
- `Gautam Jeyaraman <https://github.com/gautamjeyaraman>`_
- Minor code fixes and patches
+
+- `DhilipSiva <https://github.com/dhilipsiva>`_
+
+ - Minor patches
View
2  cyclone/web.py
@@ -1087,7 +1087,7 @@ def removeslash(method):
@functools.wraps(method)
def wrapper(self, *args, **kwargs):
if self.request.path.endswith("/"):
- if self.request.method in ("GET", "HEAD"):
+ if self.request.method in ("GET", "HEAD", "POST", "PUT", "DELETE"):
uri = self.request.path.rstrip("/")
if uri: # don't try to redirect '/' to ''
if self.request.query:
Something went wrong with that request. Please try again.