-
Notifications
You must be signed in to change notification settings - Fork 530
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
Handle Unknown Exceptions, Process Crashes and provide respective errors with GraphQL #1219
Comments
There's also an issue with queries/mutations over WebSockets, where it falls into a reconnection loop. It could be patched on the client (absinthe-graphql/absinthe-socket#57), but returning proper GraphQL errors from the server is a better solution. |
@yurtsiv server should send proper errors. |
I came to create a similar issue before finding this one, mostly because the docs on the subject ask for contributions: https://hexdocs.pm/absinthe/1.7.6/errors.html 😄 I asked for opinions in the Elixir Slack channel for Absinthe and a few links were shared:
My question to the maintainers: should Absinthe "provide a formal way to handle unexpected errors" as the OP believes? If not, what sort of guidance should live on the docs "errors" page? cc @maartenvanvliet @bruce since y'all wrote the original requests for assistance 2 and 7 years ago. 😄 Regardless of the answer, I'd like to assist! |
If submitting a bug, please provide the following:
Environment
Expected behavior
Absinthe should provide a formal way to handle unexpected errors, exceptions or process crashes(for example inside
DataLoader.KV
)and allow customized errors returned to the GraphQL client.
Actual behavior
Currently, when something goes wrong, we get phoenix default ErrorView. This is bad for even development environments.
GraphQL server should be able to properly reply with a JSON response when some error occurs.
There are community-based blog posts where that tackle this issue
But these blog posts or methods don't work when a process crashes, for example
DataLoader.KV
failed due to some issue and crashes the processI believe this part is crucial for a GraphQL server, I spent hours and days on this issue until I found the above blog posts, but they don't cover the above use cases.
It would be awesome to have some official support to tackle these situations much more swiftly as I believe this is a common problem when working with Absinthe in a production environment
Relevant Schema/Middleware Code
The text was updated successfully, but these errors were encountered: