graphile / crystal-pre-merge

Repository moved
https://github.com/graphile/crystal
39 stars 10 forks source link

Send GraphQLError to client during processGraphQLRequestBody hook #491

Closed Dodobibi closed 9 months ago

Dodobibi commented 10 months ago

If a SafeError is raised in the execution of the GraphQLProcessRequestBody hook, this error must be returned to the client (web request or ws subscription), as well as stopping the query execution flow. Especially in subscription-ws, where the connection is closed abruptly.

benjie commented 10 months ago

Rebased on latest main (with all the dependency updates from #496)

benjie commented 9 months ago

Migrated to https://github.com/graphile/crystal/pull/1781