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

Add option to validate the body as a json #206

Closed

Conversation

rodrigowirth
Copy link

This is a proposal for the issue #122

@dougwilson dougwilson self-assigned this Nov 17, 2016
@dougwilson dougwilson added the pr label Nov 17, 2016
@dougwilson
Copy link
Contributor

Hi @rodrigowirth it is not clear what this PR is even doing, even reading the docs and source. Can you give a much, much more detailed explanation of exactly what this PR does? It looks like, to me, all it does is prepend a word to the error message with the use of a switch?

@dougwilson
Copy link
Contributor

Oh, nevermind, I see the res change. Unfortunately, no, this PR is not going to be accepted. We ujse the standard error handing from Express and middleware; the module is never going to have functionality to write to the res object.

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.

2 participants