kadirahq / lokka-transport-http

HTTP Transport Layer for Lokka
MIT License
41 stars 38 forks source link

Should now include the whole response when GraphQL returns something other than 200 #29

Open mourasman opened 6 years ago

mourasman commented 6 years ago

Hey guys,

I've been using your lib recently and I've had the need to enhance the Error that is thrown when a GraphQL request returns something other than a 200. Previously, it was only throwing an Error with a generic message, but for testing purposes I needed it to include the whole response, as well, so that I could validate our application's error handling. I believe this may be a useful addition to this project, hence this PR.

If you think this is useful, feel free to merge it.

Cheers, Mourasman