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

[patch] Fix incorrect handling of Error objects in handleError() and add test… #3645

Merged
merged 1 commit into from Mar 9, 2016

Conversation

ghost
Copy link

@ghost ghost commented Mar 9, 2016

Fix incorrect handling of Error objects in handleError() and add test to prevent regression of same. With this change the error response created by the default response hooks will include more information when an error is encountered while processing a request. The body of the HTTP response will include a message and stack (if not in production.)

… to prevent regression of same. With this change the error response created by the default response hooks will include more information when an error is encountered while processing a request. The body of the HTTP response will include a message and stack (if not in production.)
@sailsbot
Copy link

sailsbot commented Mar 9, 2016

Thanks for posting, @tapuzzo-fsi! We'll look into this ASAP.

@mikermcneil
Copy link
Member

@tapuzzo-fsi awesome, thank you!

mikermcneil added a commit that referenced this pull request Mar 9, 2016
[patch] Fix incorrect handling of Error objects in handleError() and add test…
@mikermcneil mikermcneil merged commit 8b506a2 into balderdashy:master Mar 9, 2016
@mikermcneil mikermcneil added the patch This label is deprecated. Please don't use it anymore label Mar 9, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch This label is deprecated. Please don't use it anymore
Development

Successfully merging this pull request may close these issues.

2 participants