Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added code to hide unauthenticated and disallowed endpoints #68

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jfinkels
Copy link
Owner

@jfinkels jfinkels commented May 4, 2012

Fixes issue #60.

There is a problem with this code because Flask doesn't allow blueprint-local routing for errors. See the comment under flask.Blueprint.errorhandler.

@jfinkels
Copy link
Owner Author

This depends on the resolution of pallets/flask#503.

@jfinkels
Copy link
Owner Author

jfinkels commented Mar 7, 2016

This should be fixable since the linked issue has been fixed in Flask version 1.0.

@jfinkels
Copy link
Owner Author

This pull request is up-to-date, but depends on Flask 1.0 being released!

@jfinkels
Copy link
Owner Author

The update seems to be in Flask 0.11.

This adds the keyword argument `hide_disallowed_endpoints` to
`APIManager.create_api`, which allows responding with a 404 response
instead of a 405 response for requests to endpoints with a disallowed
HTTP method.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant