From: DW Talton <dtalton@contractor.linuxfoundation.org>
Date: Wed, 29 Jan 2020 21:59:59 +0000 (-0700)
Subject: Remove extra request error checking
X-Git-Tag: v0.30.1^0
X-Git-Url: https://gerrit.linuxfoundation.org/infra/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F90%2F62990%2F1;p=releng%2Flftools.git

Remove extra request error checking

Remove additional requests error checking. Current lftools needs
the ability to continue despite a http session error.

Signed-off-by: DW Talton <dtalton@contractor.linuxfoundation.org>
Change-Id: Icad82ce16f7df7466e43be7e9035abac3fe63033
---

diff --git a/lftools/api/client.py b/lftools/api/client.py
index 0e00df27..6173f59a 100644
--- a/lftools/api/client.py
+++ b/lftools/api/client.py
@@ -56,24 +56,6 @@ class RestApi(object):
         if resp.status_code == 409:
             return resp
 
-        # otherwise abort on any actual HTTP errors and suppress traceback
-        try:
-            resp.raise_for_status()
-        except requests.exceptions.RequestException as e:
-            raise e
-        except requests.exceptions.HTTPError as e:
-            raise e.args
-        except requests.exceptions.ConnectionError as e:
-            raise e
-        except requests.exceptions.ProxyError as e:
-            raise e
-        except requests.exceptions.Timeout as e:
-            raise e
-        except requests.exceptions.URLRequired as e:
-            raise e
-        except requests.exceptions.InvalidURL as e:
-            raise e
-
         if resp.text:
             try:
                 if "application/json" in resp.headers["Content-Type"]:
@@ -83,7 +65,6 @@ class RestApi(object):
                     body = resp.text
             except ValueError:
                 body = None
-
         else:
             body = None
             return resp