docs: add guide on graphql-http errors#2006
Conversation
|
@sarahxsanders is attempting to deploy a commit to the The GraphQL Foundation Team on Vercel. A member of the Team first needs to authorize it. |
benjie
left a comment
There was a problem hiding this comment.
This is really good general reference that should be applicable to any server implementing GraphQL-over-HTTP (not just the reference implementation).
One concern is it does not incorporate the application/graphql-response+json media type. Since the spec is currently a draft I wouldn't normally be too concerned, except we do link to the spec so we should recommend its recommendations.
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Co-authored-by: Benjie <benjie@jemjie.com>
Co-authored-by: Benjie <benjie@jemjie.com>
Co-authored-by: Benjie <benjie@jemjie.com>
Co-authored-by: Benjie <benjie@jemjie.com>
Co-authored-by: Benjie <benjie@jemjie.com>
Co-authored-by: Benjie <benjie@jemjie.com>
Co-authored-by: Benjie <benjie@jemjie.com>
Co-authored-by: Benjie <benjie@jemjie.com>
added a section about understanding response formats, I hope that addresses this better! |
enisdenjo
left a comment
There was a problem hiding this comment.
Are we sure about the 415 and 422? The spec itself does not recommend them, it's up to the implementor to choose using it and follow HTTP conventions.
|
@enisdenjo I removed those sections and added a section w/ a disclaimer, let me know if this addresses that! |
|
It's looking good now, thanks! |
Description
graphql-httperrors (just some common ones) and how to debug themFeel free to suggest additional common errors if there are any that might be important to highlight here!