Merge pull request ozgur#7 from ozgur/last_error_impl

last error is persisted for the authentication

Author: ozgur

linkedin/linkedin.py

@@ -63,6 +63,7 @@ def __init__(self, key, secret, redirect_uri, permissions=[]):
         self.state = None
         self.authorization_code = None
         self.token = None
+        self._error = None
 
     @property
     def authorization_url(self):
@@ -77,6 +78,10 @@ def authorization_url(self):
         qsl = ['%s=%s' % (urllib.quote(k), urllib.quote(v)) for k, v in qd.items()]
         return '%s?%s' % (self.AUTHORIZATON_URL, '&'.join(qsl))
 
+    @property
+    def last_error(self):
+        return self._error
+
     def get_new_state(self):
         return hashlib.md5(
             '%s%s' % (random.randrange(0, 2**63), self.secret)).hexdigest()
@@ -96,6 +101,7 @@ def get_access_token(self, timeout=60):
             raise LinkedInHTTPError(error.message)
         else:
             if 'error' in response:
+                self._error = response['error_description']
                 raise LinkedInError(response)
         self.token = AccessToken(response['access_token'], response['expires_in'])
         return self.token
@@ -156,7 +162,7 @@ def get_profile(self, member_id=None, member_url=None, selectors=None,
         except requests.ConnectionError as error:
             raise LinkedInHTTPError(error.message)
         else:
-            if 'error' in response:
+            if self.request_succeeded(response):
                 raise LinkedInError(response)
             return response
 
@@ -172,7 +178,7 @@ def search_profile(self, selectors=None, params=None, headers=None):
         except requests.ConnectionError as error:
             raise LinkedInHTTPError(error.message)
         else:
-            if 'error' in response:
+            if self.request_succeeded(response):
                 raise LinkedInError(response)
             return response