Skip to content
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.

Sign up for GitHub

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

Jump to bottom

GraphQLWS: store the RoutingContext in GraphQLContext #2476

Closed
tsegismont opened this issue Aug 29, 2023 · 1 comment
Closed

GraphQLWS: store the RoutingContext in GraphQLContext #2476

tsegismont opened this issue Aug 29, 2023 · 1 comment
Assignees
@tsegismont
Labels
enhancement graphql Issues related to GraphQL module
Milestone
4.4.5

Comments

@tsegismont
Copy link
Contributor

This can be useful when, e.g. the webapp is protected with authentication.

This was already possible with the GraphQLHandler:

vertx-web/vertx-web-graphql/src/main/java/io/vertx/ext/web/handler/graphql/impl/GraphQLHandlerImpl.java

Line 414 in e826941

builder.graphQLContext(Collections.singletonMap(RoutingContext.class, rc));

@tsegismont tsegismont added this to the 4.4.5 milestone Aug 29, 2023
@tsegismont tsegismont self-assigned this Aug 29, 2023
@tsegismont
Copy link
Contributor Author

Fixed by a2ee6c8

@tsegismont tsegismont added the graphql Issues related to GraphQL module label Aug 29, 2023
This was referenced Aug 29, 2023
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsegismont tsegismont

Labels
enhancement graphql Issues related to GraphQL module
Milestone
4.4.5
Development

No branches or pull requests
1 participant
@tsegismont