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

handle empty bodied response #237

Closed
wants to merge 1 commit into from
Closed

handle empty bodied response #237

wants to merge 1 commit into from

Conversation

ephur
Copy link

@ephur ephur commented Nov 1, 2013

Sometimes the response body doesn't contain anything, handle this error
condition by retrying, and remaining within the constraints of the
timeout.

I also relocated the timeout logic above the check to 'resp_body' == "ERROR" because it could timeout and resp_body could still be None.

Sometimes the response body doesn't contain anything, handle this error
condition by retrying, and remaining within the constraints of the
timeout.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling ec71eb5 on ephur:master into 2c431e0 on rackspace:master.

EdLeafe added a commit that referenced this pull request Nov 13, 2013
@EdLeafe
Copy link
Contributor

EdLeafe commented Nov 13, 2013

Thanks for the PR. I merged this manually into the 'working' branch, as 'master' is reserved for released code.

@EdLeafe EdLeafe closed this Nov 13, 2013
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