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

Fix GraphQL query with variables to HTTP body request #1223

Conversation

jcamiel
Copy link
Collaborator

@jcamiel jcamiel commented Jan 29, 2023

Fix construction of the POST body request when GraphQL query has variables (see https://graphql.org/learn/serving-over-http/#post-request)

@jcamiel jcamiel linked an issue Jan 29, 2023 that may be closed by this pull request
@jcamiel jcamiel force-pushed the 1218-graphql-variables-serialize-as-a-string-and-not-json branch from 8da413e to ba1c48c Compare January 29, 2023 17:55
@jcamiel jcamiel force-pushed the 1218-graphql-variables-serialize-as-a-string-and-not-json branch from ba1c48c to 98f5db3 Compare January 29, 2023 18:05
@jcamiel jcamiel marked this pull request as ready for review January 30, 2023 13:30
@fabricereix
Copy link
Collaborator

/accept

@hurl-bot
Copy link
Collaborator

🕗 /accept is running, please wait for completion.

@hurl-bot
Copy link
Collaborator

✅ Pull request accepted and closed by fabricereix with fast forward merge..

# List of commits merged from Orange-OpenSource/hurl/1218-graphql-variables-serialize-as-a-string-and-not-json branch into Orange-OpenSource/hurl/master branch:

  • 98f5db3 Fix GraphQL query with variables to HTTP body request

@hurl-bot hurl-bot closed this Jan 30, 2023
@hurl-bot hurl-bot deleted the 1218-graphql-variables-serialize-as-a-string-and-not-json branch January 30, 2023 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

graphql variables serialize as a string and not json
3 participants