We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Stepping through the code and it seems that the same OperationContext is added twice to the active context.Context. This happens just before the call to exec.DispatchOperation at: https://github.com/99designs/gqlgen/blob/master/graphql/handler/transport/http_post.go#L51
then at the top of exec.DispathcOperation at: https://github.com/99designs/gqlgen/blob/master/graphql/executor/executor.go#L90
I expected to see only one call to graphql.WithOperationContext(). I could be wrong but it seems there is a redundant call.
gqlgen version
go version
The text was updated successfully, but these errors were encountered:
@lwc close this issue, it its fixed.
Sorry, something went wrong.
@StevenACoffman please close issue, it's fixed via #1641
No branches or pull requests
What happened?
Stepping through the code and it seems that the same OperationContext is added twice to the active context.Context. This happens just before the call to exec.DispatchOperation at:
https://github.com/99designs/gqlgen/blob/master/graphql/handler/transport/http_post.go#L51
then at the top of exec.DispathcOperation at:
https://github.com/99designs/gqlgen/blob/master/graphql/executor/executor.go#L90
What did you expect?
I expected to see only one call to graphql.WithOperationContext(). I could be wrong but it seems there is a redundant call.
Minimal graphql.schema and models to reproduce
versions
gqlgen version
? v0.13.0go version
? go version go1.16.6 windows/amd64The text was updated successfully, but these errors were encountered: