Log full error data when login fails in provisioning API

mautrix · Jul 14, 2023 · 352d410 · 352d410
1 parent 231cb10
commit 352d410
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/mautrix_facebook/web/public.py b/mautrix_facebook/web/public.py
@@ -272,7 +272,7 @@ async def login(self, request: web.Request) -> web.Response:
             )
         except OAuthException as e:
             track(user, "$login_failed", {"error": str(e)})
-            self.log.debug(f"Got OAuthException {e} for {user.mxid}")
+            self.log.debug(f"Got OAuthException {e} for {user.mxid}: {e.data}")
             return web.json_response({"error": str(e)}, headers=self._acao_headers, status=401)
 
     async def login_2fa(self, request: web.Request) -> web.Response:
@@ -320,7 +320,7 @@ async def login_2fa(self, request: web.Request) -> web.Response:
             )
         except OAuthException as e:
             track(user, "$login_failed", {"error": str(e)})
-            self.log.debug(f"Got OAuthException {e} for {user.mxid} in 2fa stage")
+            self.log.debug(f"Got OAuthException {e} for {user.mxid} in 2fa stage: {e.data}")
             return web.json_response({"error": str(e)}, headers=self._acao_headers, status=401)
 
     async def login_approved(self, request: web.Request) -> web.Response:
@@ -345,7 +345,9 @@ async def login_approved(self, request: web.Request) -> web.Response:
             return web.json_response({"status": "logged-in"}, headers=self._acao_headers)
         except OAuthException as e:
             track(user, "$login_failed", {"error": str(e)})
-            self.log.debug(f"Got OAuthException {e} for {user.mxid} in checkpoint login stage")
+            self.log.debug(
+                f"Got OAuthException {e} for {user.mxid} in checkpoint login stage: {e.data}"
+            )
             return web.json_response({"error": str(e)}, headers=self._acao_headers, status=401)
 
     async def login_check_approved(self, request: web.Request) -> web.Response: