Include response body in 'unexpected status' error message. about graphql HOT 3 CLOSED

Thanks for the issue report @cjwagner, and for providing the details.

This issue is already on my radar, as PR #24 was sent to address a similar need. See discussion there for some background and #24 (comment) for latest status. The details you're provided in the issue should help find a viable resolution.

I'd prefer to think about this and discuss it in the issue first. Once we have agreement on a resolution plan, implementing the fix will be the easier part.

from graphql.

Here is an example: Something went wrong while executing your query. This may be the result of a timeout, or it could be a GitHub bug. Please include 'EA06:8FDC:2D9BD3C:56E05DD:599AFE05' when reporting this issue.

The solution shouldn't be specific to the format or contents of the response body though since this is just an example from GitHub. The %q format token is more appropriate than %s or "%s" here as it will clearly indicate the start and end of the contents by escaping any "s.

from graphql.

Ok, I think adding a ReadAll on the response body and including it in the error text is a reasonable first step to make, for now. Let's do that.

we are seeing lots of 502 and 403 errors that should include explanations in the response bodies.

I'm not sure whether to use %s or %q to format the response body. @cjwagner, would you be able to share sample response bodies during those errors, if you have them available?

A bigger step might be to create a struct error type with a ResponseBody []byte field in it, but I'd rather defer that until it's really needed.

I've sent PR #30, we can discuss the details there.

from graphql.