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

Dump all responses, before bailing out for non-successful http status codes #553

Merged
merged 1 commit into from
Jul 18, 2017
Merged

Dump all responses, before bailing out for non-successful http status codes #553

merged 1 commit into from
Jul 18, 2017

Conversation

nvartolomei
Copy link
Contributor

No description provided.

@connor4312
Copy link
Contributor

Any chance of getting this in @olivere? This is a simple change that makes debugging errors a whole lot easier! 😄

@olivere
Copy link
Owner

olivere commented Jul 18, 2017

I'm busy at work. I will try to spend an afternoon on all open issues later this week.

@olivere olivere merged commit 8f4f2ff into olivere:release-branch.v5 Jul 18, 2017
olivere added a commit that referenced this pull request Jul 18, 2017
This closes long standing issues #297 and #553.
@olivere
Copy link
Owner

olivere commented Jul 18, 2017

Will be part of 5.0.42 later today.

@connor4312
Copy link
Contributor

Awesome, thanks!

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.

3 participants