From: Kyle <kyle.rockman@mac.com>
Date: Mon, 27 Jan 2014 20:01:11 +0000 (-0600)
Subject: logging fix for invalid response
X-Git-Tag: v0.2.23~50^2~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=244f5988056a54050129b06a03f9961af794f767;p=tools%2Fpython-jenkinsapi.git

logging fix for invalid response
---

diff --git a/jenkinsapi/jenkinsbase.py b/jenkinsapi/jenkinsbase.py
index bc5624b..eadfc6e 100644
--- a/jenkinsapi/jenkinsbase.py
+++ b/jenkinsapi/jenkinsbase.py
@@ -6,7 +6,6 @@ import ast
 import logging
 from jenkinsapi import config
 from jenkinsapi.custom_exceptions import JenkinsAPIException
-log = logging.getLogger(__name__)
 
 
 class JenkinsBase(object):
@@ -61,10 +60,12 @@ class JenkinsBase(object):
     def get_data(self, url, params=None):
         requester = self.get_jenkins_obj().requester
         response = requester.get_url(url, params)
+        if response.status_code != 200 :
+            response.raise_for_status()
         try:
             return ast.literal_eval(response.text)
         except Exception:
-            log.exception('Inappropriate content found at %s', url)
+            logging.exception('Inappropriate content found at %s', url)
             raise JenkinsAPIException('Cannot parse %s' % response.content)
 
     @classmethod