Merge d4216a4 into 4ae0014

tests/test_functional.py

@@ -22,6 +22,10 @@ def test_redirect(self, app):
         res = app.get("/foobar/18", status=303)
         assert res.status == "303 See Other"
 
+    def test_redirect_not_allowed(self, app):
+        res = app.post("/foobar/18", status=405)
+        assert res.status == "405 Method Not Allowed"
+
     def test_redirect_no_match(self, app):
         res = app.get("/test", status=404)
         assert res.status == "404 Not Found"

urihandler/views.py

@@ -1,12 +1,13 @@
 from pyramid.httpexceptions import HTTPBadRequest
+from pyramid.httpexceptions import HTTPMethodNotAllowed
 from pyramid.httpexceptions import HTTPNotFound
 from pyramid.httpexceptions import HTTPSeeOther
 from pyramid.view import view_config
 
 from urihandler.utils import create_version_hash
 
 
-@view_config(route_name="redirect")
+@view_config(route_name="redirect", request_method=("GET", "HEAD", "OPTIONS"))
 def redirect(request):
     uri = request.host_url + "/" + request.matchdict["uri"]
     redirect = request.uri_handler.handle(uri, request)
@@ -15,6 +16,11 @@ def redirect(request):
     return HTTPSeeOther(redirect)
 
 
+@view_config(route_name="redirect")
+def redirect_not_allowed(request):
+    raise HTTPMethodNotAllowed()
+
+
 @view_config(route_name="handle")
 def handle(request):
     uri = request.params.get("uri", None)