graphql-go / graphql-go/graphql

Empty OperationName should not be coerced to “” (but null)

Open
#616 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
10.1k
Forks
845
PR merge metrics
No merged PRs in 30d

Description

Hi!
We uncovered a compatibility issue with the specification, while calling out express based grapqhQL servers.

The OperationName, when not specified is always coerced to an empty string, which doesn’t appear to be compliant with the specification: https://spec.graphql.org/draft/#sec-Executing-Requests

Initially, I sought to fix the compatibility in the graphQL express implementation, by allowing empty strings as empty operation names through https://github.com/graphql/express-graphql/pull/768.

The maintainers raised that the spec’s requires empty operation names to be conveyed by a null value.

Thanks!

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.