- Notifications
You must be signed in to change notification settings - Fork358
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Gh-3353: Ensure user info passed in GafferPop#3354
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@## develop #3354 +/- ##=============================================- Coverage 68.08% 68.06% -0.03% Complexity 2596 2596 ============================================= Files 957 957 Lines 30670 30674 +4 Branches 3399 3402 +3 =============================================- Hits 20881 20877 -4- Misses 8304 8313 +9+ Partials 1485 1484 -1 ☔ View full report in Codecov by Sentry. |
rest-api/spring-rest/src/main/java/uk/gov/gchq/gaffer/rest/controller/GremlinController.javaShow resolvedHide resolved
rest-api/spring-rest/src/main/java/uk/gov/gchq/gaffer/rest/handler/GremlinWebSocketHandler.javaShow resolvedHide resolved
p29876 previously approved these changesJan 20, 2025
cn337131 previously approved these changesJan 21, 2025
cn337131 approved these changesJan 21, 2025
p29876 approved these changesJan 21, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Addresses some of the issues with passing information from the REST API to gafferpop by ensuring each request now has its own instance of the tinkerpop graph layer so the variables are unique per request. Also stops the user from being reset immediately by the custom GraphStep.
Few other minor bits of tidying and improvements to otel logging.
Related issue