From 36bc38406a8059a541b6974d415c8d422498d4d9 Mon Sep 17 00:00:00 2001 From: "sergeyberezin@chromium.org" Date: Tue, 25 Feb 2014 22:36:13 +0000 Subject: [PATCH] Handle SSL timeouts in Rietveld requests. BUG=346845 R=maruel@chromium.org Review URL: https://codereview.chromium.org/180273003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/depot_tools@253271 0039d316-1c4b-4281-b951-d872f2087c98 --- rietveld.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/rietveld.py b/rietveld.py index 39cb19a9d..9dfe8892b 100644 --- a/rietveld.py +++ b/rietveld.py @@ -18,6 +18,7 @@ import copy import json import logging import re +import ssl import time import urllib2 @@ -405,6 +406,11 @@ class Rietveld(object): not 'EOF occurred in violation of protocol' in e.reason): # Usually internal GAE flakiness. raise + except ssl.SSLError, e: + if retry >= (maxtries - 1): + raise + if not 'timed out' in e.reason: + raise # If reaching this line, loop again. Uses a small backoff. time.sleep(1+maxtries*2) finally: