Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Handle SSL errors with retries #159

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
Closed
wants to merge 1 commit into from
Closed

Handle SSL errors with retries #159

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Dec 15, 2015

This was already reviewed once at https://codereview.chromium.org/1522953002.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 15, 2015
@ghost
Copy link
Author

ghost commented Dec 15, 2015

This is going to address issues #71, #97, #158.

nathanielmanistaatgoogle
nathanielmanistaatgoogle reviewed Dec 15, 2015
View reviewed changes
googleapiclient/http.py
@@ -546,14 +547,21 @@ def next_chunk(self, num_retries=0):
}
http = self._request.http

resp = None

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@nathanielmanistaatgoogle
Copy link
Contributor

Thank you for the contribution!

@googlebot
Copy link
Collaborator

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@googlebot googlebot added cla: no This human has *not* signed the Contributor License Agreement. and removed cla: yes This human has signed the Contributor License Agreement. labels Dec 15, 2015
@ghost
Copy link
Author

ghost commented Dec 15, 2015

I've been using a mixture of accounts to create CLs and this pull request, which seems to have confused the CLA check. I'll re-create the pull request from the same account as the author.

@ghost ghost closed this Dec 15, 2015
@rryk rryk mentioned this pull request Dec 15, 2015
Merged
akrherz pushed a commit to akrherz/google-api-python-client that referenced this pull request Apr 1, 2019
@craigcitro
Merge pull request googleapis#159 from dhermes/speedup-openssl-import
22a532d 
Deferring OpenSSL import until usage.
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla: no This human has *not* signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nathanielmanistaatgoogle @googlebot @rryk