When raise exception I got graphql error

[Bunlong]

I just move from using responder library to starlette. I got the graphql error when I raise exception.

Server:

app = Starlette(debug=True, routes=[
    Route('/', show_version),
    Route('/healthz', check_health),
    Route('/graphql', GraphQLApp(schema=schema)),
])

if __name__ == '__main__':
    uvicorn.run(app, host='0.0.0.0', port=9090)

Schema:

...

raise Exception('Something wrong')

...

Error:

What I expect is it return error message Something wrong with the http status 200 instead of error above. Thanks!

[chrisbrake]

I've run into this issue as well.

This appears to be the code responsible.

starlette/starlette/graphql.py

Lines 112 to 118 in e430706

	status_code = (
	status.HTTP_400_BAD_REQUEST if result.errors else status.HTTP_200_OK
	)
	return JSONResponse(
	response_data, status_code=status_code, background=background
	)

Looks to be more or less original to the implementation here, it came along with this commit: 315a21b.

Maintainers, would an acceptable solution be to add an option to specify the status code to use when errors exist?
With the default being the current behavior of course.

[JayH5]

Thank you for filing this issue. We have decided to deprecate GraphQL support within Starlette itself so I am going to close this issue. See #619.