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

Validate Defaults #3

Open
raphaelhetzel opened this issue Jan 24, 2017 · 1 comment
Open

Validate Defaults #3

raphaelhetzel opened this issue Jan 24, 2017 · 1 comment

Comments

@raphaelhetzel
Copy link
Contributor

The defaults are currently not validated,
what could lead to errors especially in the case of the sort options or other parsers with more complex output.
This could be verified but as the defaults are defined in code and this would also slow down the handler,
i'm not sure how whether this should be done. A different solution would be to provide a way to easily spec this (Shared Examples?).

@andreaseger
Copy link
Member

hmm maybe we can verify the defaults once during definition. That way we have both benefits no overhead for the request parsing and validated default options

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

No branches or pull requests

2 participants