Skip to content

Allow batch handler to set http status #156

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

Conversation

jmccaull
Copy link
Contributor

@jmccaull jmccaull commented Mar 1, 2019

@oliemansm, another minor change - we found if we throw an exception the server will return 400, but if we try to write any sort of error message tomcat will not update the status preventing us from sending back 400 with any sort of message

@jmccaull
Copy link
Contributor Author

jmccaull commented Mar 1, 2019

I'm going to update the test to cover this case as well

@oliemansm oliemansm merged commit ff6b4f5 into graphql-java-kickstart:master Mar 2, 2019
@oliemansm
Copy link
Member

@jmccaull Version 7.3.3 has been released.

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.

2 participants