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.

Sign up for GitHub

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

Jump to bottom

Avoid panicking, handle error #72

Merged

mweagle merged 1 commit into mweagle:master from nylar:dont-panic

Oct 5, 2017

Contributor

nylar commented Oct 5, 2017

Handle error from http.NewRequest() and avoid panicking if client.Do() should fail.


          Avoid panicking, handle error

d3ce8f0

Owner

mweagle commented Oct 5, 2017

LGTM - thanks @nylar .

mweagle merged commit ea7ce50 into mweagle:master

nylar deleted the dont-panic branch

October 5, 2017 10:10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet