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.

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

Avoid graphql.AssertException for persisted query #930

Closed
rstoyanchev opened this issue Mar 20, 2024 · 0 comments
Closed

Avoid graphql.AssertException for persisted query #930

rstoyanchev opened this issue Mar 20, 2024 · 0 comments
Assignees
Labels
type: enhancement A general enhancement
Milestone

Comments

@rstoyanchev
Copy link
Contributor

For persisted queries, the request could look like this:

 {
     "extensions":{
        "persistedQuery":{
           "version":1,
           "sha256Hash":"ecf4edb46db40b5132295c0291d62fb65d6759a9eedfa4d5d612dd5ec54a6b38"
          }
      }
}

As there is no "query", setting the query on ExecutionInput results in a "query can't be null" AssertException.

We should use PersistedQuerySupport#PERSISTED_QUERY_MARKER as the query in that case.

@rstoyanchev rstoyanchev added the type: enhancement A general enhancement label Mar 20, 2024
@rstoyanchev rstoyanchev added this to the 1.2.6 milestone Mar 20, 2024
@rstoyanchev rstoyanchev self-assigned this Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

No branches or pull requests

1 participant