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

Respecting HTTP status code #3

Merged
merged 2 commits into from
Oct 16, 2013
Merged

Conversation

hirnschmalz
Copy link

No description provided.

hirnschmalz added 2 commits October 15, 2013 20:32
Checking HTTP error codes and cancel download if HTTP status Code >= 400
@thibaultcha
Copy link
Owner

Nice, thank you. Now I think I will add some unit tests on this lib soon, because those things need to be tested :)

Congrats on your first PR 👏

thibaultcha added a commit that referenced this pull request Oct 16, 2013
Watching HTTP error status codes
@thibaultcha thibaultcha merged commit 76f2f1c into thibaultcha:master Oct 16, 2013
thibaultcha added a commit that referenced this pull request Oct 25, 2013
Watching HTTP error status codes
@thibaultcha
Copy link
Owner

Hi,

After some struggling with Git, I had to rebase commits to fix the authors of some commits, including yours. I fixed it but I need your Github email address so GitHub can display your profile in the "contributors" section of this project. Would you consider send me your email at [email protected] or here? (I will delete it afterwards). Sorry.

Thank you,

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.

2 participants