Repository navigation
Conversation
Contributor
Test ReportTest Results
Code Coverage (Java 25)
Changed Class Coverage (2 classes)
|
This branch has not been deployed
This file contains hidden or 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
Sign up for free
to 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.
Spock, ApolloPersistedQuerySupportTest on master:
PersistedQuerySupport.getDocumentAsynconly callspersistedQueryIdIsInvalidfrom the cache-miss callback. Once a hash is cached, any request carrying it runs the cached document whatever query text it sends:{ public }sent with sha256 of{ secret }executes{ secret }with no error, whileExecutionInput.getQuery()(what instrumentation, logging and the execution id provider see) still says{ public }. Apollo's server rejects this case with "provided sha does not match query", and #2395 asked for the same error whenever both a hash and a query are present.The check now runs before the cache lookup when real query text is sent (not blank, not
PERSISTED_QUERY_MARKER). The basepersistedQueryIdIsInvalidstill returns false, so onlyApolloPersistedQuerySupportbehaves differently. The existing "hash in preference to query text" test asserted the old behaviour, so it now sends matching text, and new cases cover the mismatch at the unit level and end to end throughGraphQL.