[Update] 添加log信息

Author: ibuler

jms/__init__.py

@@ -2,4 +2,4 @@
 # -*- coding: utf-8 -*-
 #
 
-__version__ = '0.0.55'
+__version__ = '0.0.56'

jms/request.py

@@ -9,9 +9,11 @@
 
 from .exception import RequestError, ResponseError, RegisterError
 from .url import API_URL_MAPPING
+from .utils import get_logger
 
 _USER_AGENT = 'jms-sdk-py'
 CACHED_TTL = os.environ.get('CACHED_TTL', 30)
+logger = get_logger(__file__)
 
 
 class HttpRequest(object):
@@ -69,12 +71,16 @@ def set_header(self, k, v):
     @staticmethod
     def clean_result(resp):
         if resp.status_code >= 500:
-            raise ResponseError("Response code is {0.status_code}: {0.text}".format(resp))
+            msg = "Response code is {0.status_code}: {0.text}".format(resp)
+            logger.error(msg)
+            raise ResponseError(msg)
 
         try:
             _ = resp.json()
         except (json.JSONDecodeError, simplejson.scanner.JSONDecodeError):
-            raise ResponseError("Response json couldn't be decode: {0.text}".format(resp))
+            msg = "Response json couldn't be decode: {0.text}".format(resp)
+            logger.error(msg)
+            raise ResponseError(msg)
         else:
             return resp
 
@@ -99,14 +105,18 @@ def do(self, api_name=None, pk=None, method='get', use_auth=True,
                           **kwargs)
         if use_auth:
             if not self.auth:
-                raise RequestError('Authentication required')
+                msg = 'Authentication required, but not provide'
+                logger.error(msg)
+                raise RequestError(msg)
             else:
                 self.auth.sign_request(req)
 
         try:
             resp = req.do()
         except (requests.ConnectionError, requests.ConnectTimeout) as e:
-            raise RequestError("Connect endpoint {} error: {}".format(self.endpoint, e))
+            msg = "Connect endpoint {} error: {}".format(self.endpoint, e)
+            logger.error(msg)
+            raise RequestError(msg)
 
         return self.clean_result(resp)
 

jms/service.py

@@ -75,7 +75,7 @@ def set_auth(self):
 
     def valid_auth(self):
         delay = 1
-        while delay < 10:
+        while delay < 100:
             user = self.get_profile()
             if not user:
                 msg = "Connect server error or access key is invalid, " \

jms/users.py

@@ -88,6 +88,8 @@ def get_profile(self):
         except (RequestError, ResponseError):
             return None
 
+        if resp.status_code == 403:
+            logger.error("Error code 403, permission deny, access key error")
         user = User.from_json(resp.json())
         return user