Added code to hide unauthenticated and disallowed endpoints#68
Open
jfinkels wants to merge 4 commits into
Open
Added code to hide unauthenticated and disallowed endpoints#68jfinkels wants to merge 4 commits into
jfinkels wants to merge 4 commits into
Conversation
Owner
Author
|
This depends on the resolution of pallets/flask#503. |
jfinkels
force-pushed
the
hideendpoints
branch
from
7355c4b to
171cb55
Compare
Owner
Author
|
This should be fixable since the linked issue has been fixed in Flask version 1.0. |
Owner
Author
|
This pull request is up-to-date, but depends on Flask 1.0 being released! |
jfinkels
force-pushed
the
hideendpoints
branch
from
df82bcd to
85e56d4
Compare
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.