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

Exception Handling for requests module. #16

Closed
wants to merge 8 commits into from

Conversation

jayanthchandra
Copy link

There is no current exception handling for requests module as a result if it breaks in fetch the whole program gets crashed.This would resolve this issue.

@jayanthchandra jayanthchandra changed the title Addition of Try/Except for requests Exception Handling for requests module. Jul 30, 2019
@pypt
Copy link
Contributor

pypt commented Jul 31, 2019

Thanks @jayanthchandra, that's really nice of you!

I've fixed the bug myself in 940261b by doing some bigger refactoring plus introducing some extra tests.

Thanks anyway!

@pypt pypt closed this Jul 31, 2019
@jayanthchandra jayanthchandra deleted the fix/Issue-14 branch July 31, 2019 11:42
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