From 45b402fafcb1b1d881dc61540e9ca84e564105d7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fran=C3=A7ois=20Voron?= Date: Sat, 5 Oct 2024 09:04:38 +0200 Subject: [PATCH] Add a warning log when OAuth Provider refresh fails Will help debug https://github.com/orgs/fief-dev/discussions/447 --- fief/apps/api/routers/oauth_providers.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/fief/apps/api/routers/oauth_providers.py b/fief/apps/api/routers/oauth_providers.py index fe8073af..514e7ff1 100644 --- a/fief/apps/api/routers/oauth_providers.py +++ b/fief/apps/api/routers/oauth_providers.py @@ -17,7 +17,7 @@ from fief.dependencies.repositories import get_repository from fief.dependencies.webhooks import TriggerWebhooks, get_trigger_webhooks from fief.errors import APIErrorCode -from fief.logger import AuditLogger +from fief.logger import AuditLogger, logger from fief.models import AuditLogMessage, OAuthProvider from fief.models.oauth_account import OAuthAccount from fief.repositories import ( @@ -196,6 +196,11 @@ async def get_user_access_token( detail=APIErrorCode.OAUTH_PROVIDER_REFRESH_TOKEN_NOT_SUPPORTED, ) from e except RefreshTokenError as e: + logger.warning( + "Error while refreshing OAuth Provider access token", + message=e.message, + error_body=e.response.text if e.response is not None else None, + ) raise HTTPException( status_code=status.HTTP_400_BAD_REQUEST, detail=APIErrorCode.OAUTH_PROVIDER_REFRESH_TOKEN_ERROR,