feat: show response content when fail to get value from dict

modules/onedrive/_rewrite.py

@@ -14,69 +14,69 @@
 
 
 def authenticate(self, code, redirect_uri, client_secret, resource=None):
-        params = {
-            "client_id": self.client_id,
-            "redirect_uri": redirect_uri,
-            "client_secret": client_secret,
-            "code": code,
-            "response_type": "code",
-            "grant_type": "authorization_code"
-        }
-
-        if resource is not None:
-            params["resource"] = resource
-
-        auth_url = self._auth_token_url
-        headers = {"Content-Type": "application/x-www-form-urlencoded"}
-        response = self._http_provider.send(
-            method="POST",
-            headers=headers,
-            url=auth_url,
-            data=params
-        )
-
+    params = {
+        "client_id": self.client_id,
+        "redirect_uri": redirect_uri,
+        "client_secret": client_secret,
+        "code": code,
+        "response_type": "code",
+        "grant_type": "authorization_code",
+    }
+
+    if resource is not None:
+        params["resource"] = resource
+
+    auth_url = self._auth_token_url
+    headers = {"Content-Type": "application/x-www-form-urlencoded"}
+    response = self._http_provider.send(
+        method="POST", headers=headers, url=auth_url, data=params
+    )
+
+    try:
         rcont = json.loads(response.content)
 
-        headers = {'Authorization' : f"Bearer {rcont['access_token']}"}
+        headers = {"Authorization": f"Bearer {rcont['access_token']}"}
         response = self._http_provider.send(
-            method='GET',
-            headers=headers,
-            url=self._http_provider.base_url
+            method="GET", headers=headers, url=self._http_provider.base_url
+        )
+        username = json.loads(response.content)["userPrincipalName"]
+
+        self._session = self._session_type(
+            username=username,
+            token_type=rcont["token_type"],
+            expires_in=rcont["expires_in"],
+            scope_string=rcont["scope"],
+            access_token=rcont["access_token"],
+            client_id=self.client_id,
+            auth_server_url=self._auth_token_url,
+            redirect_uri=redirect_uri,
+            refresh_token=rcont["refresh_token"] if "refresh_token" in rcont else None,
+            client_secret=client_secret,
         )
-        username = json.loads(response.content)['userPrincipalName']
-
-        try:
-            self._session = self._session_type(
-                username=username,
-                token_type=rcont["token_type"],
-                expires_in=rcont["expires_in"],
-                scope_string=rcont["scope"],
-                access_token=rcont["access_token"],
-                client_id=self.client_id,
-                auth_server_url=self._auth_token_url,
-                redirect_uri=redirect_uri,
-                refresh_token=rcont["refresh_token"] if "refresh_token" in rcont else None,
-                client_secret=client_secret
-            )
-        except:
-            raise Exception('response content:\n' + str(rcont))
+
+    except Exception as e:
+        raise Exception(f"Exception: {e}\nresponse content: {str(rcont)}\n")
 
 
 def authenticate_request(self, request):
     if self._session is None:
-        raise RuntimeError("""Session must be authenticated 
-            before applying authentication to a request.""")
+        raise RuntimeError(
+            """Session must be authenticated 
+            before applying authentication to a request."""
+        )
 
-    if self._session.is_expired() and 'offline_access' in self.scopes:
+    if self._session.is_expired() and "offline_access" in self.scopes:
         self.refresh_token()
 
     request.append_option(
-        HeaderOption("Authorization",
-                        "bearer {}".format(self._session.access_token)))
+        HeaderOption("Authorization", "bearer {}".format(self._session.access_token))
+    )
 
 
 def create_session(self, item=None):
-    return ItemCreateSessionRequestBuilder(self.append_to_request_url("createUploadSession"), self._client, item=item)
+    return ItemCreateSessionRequestBuilder(
+        self.append_to_request_url("createUploadSession"), self._client, item=item
+    )
 
 
 def http_response_init(self, status, headers, content):
@@ -90,12 +90,13 @@ def session(self):
     return self._session
 
 
-def logout(self, username=''):
+def logout(self, username=""):
     has_other_user = self._session.logout(username)
     if not has_other_user:
         self._session = None
     return has_other_user
 
+
 # Overwrite the standard upload operation to use this one
 AuthProvider.authenticate = authenticate
 AuthProvider.authenticate_request = authenticate_request