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

defaulthandler in options of v2.x #92

Open
wants to merge 2 commits into
base: v2.x
Choose a base branch
from
Open

defaulthandler in options of v2.x #92

wants to merge 2 commits into from

Conversation

gonenduk
Copy link

@gonenduk gonenduk commented Nov 2, 2017

Add an option for a default (fallback) handler to be used if no other handler was found or given:

  • Default handler is optional.
  • Handler must be a function.
  • Default handler is a fallback and is called only if handlers (object or files) doesn't have the right handler.

NOTE 1: in version 1.x there was an option with the same name which was used not as a fallback. It was always used for all paths and methods - making it quite useless.
The new option use case might be a 501 "not implemented" error (instead of 404 we have today) or any other default behaviour.

NOTE 2: I only made the code changes. If you approve the change, I will update the readme doc as well with explanations and samples.

@gonenduk
Copy link
Author

gonenduk commented Nov 5, 2017

@subeeshcbabu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant