Fix v4.2.0 regression in variables: null
(et al)
#7203
Merged
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.
In v4.2.0 (#7171) we changed POST handling to be stricter if
operationName
,variables
, orextensions
were provided with a surprising data type. This was intended to pass more of the optional recommendations of the GraphQL Over HTTP spec as tested by the graphql-http audit suite. However, we were overzealous and also banned providing these parameters as an explicitnull
, which is documented by the spec as legitimate. (And some clients, such as FIXME, actually sendvariables: null
in practice.)We added explicit tests for this to the
graphql-http
test suite (graphql/graphql-http#28) and this commit allows thesenull
s again.Fixes #7200.