From ff639a20d01437a8ad06c6f0b5b62d61ca19d440 Mon Sep 17 00:00:00 2001 From: Timothy Jaeryang Baek Date: Sat, 28 Jun 2025 00:48:24 +0400 Subject: [PATCH] refac --- backend/open_webui/config.py | 26 ++++++++++++++++++++------ 1 file changed, 20 insertions(+), 6 deletions(-) diff --git a/backend/open_webui/config.py b/backend/open_webui/config.py index 0a4275d55b..2848d21d6f 100644 --- a/backend/open_webui/config.py +++ b/backend/open_webui/config.py @@ -434,7 +434,7 @@ OAUTH_SCOPES = PersistentConfig( OAUTH_TIMEOUT = PersistentConfig( "OAUTH_TIMEOUT", "oauth.oidc.oauth_timeout", - os.environ.get("OAUTH_TIMEOUT", 5), + os.environ.get("OAUTH_TIMEOUT", ""), ) OAUTH_CODE_CHALLENGE_METHOD = PersistentConfig( @@ -548,8 +548,12 @@ def load_oauth_providers(): server_metadata_url="https://accounts.google.com/.well-known/openid-configuration", client_kwargs={ "scope": GOOGLE_OAUTH_SCOPE.value, - "timeout": OAUTH_TIMEOUT.value - }, + **( + {"timeout": int(OAUTH_TIMEOUT.value)} + if OAUTH_TIMEOUT.value + else {} + ), + }, redirect_uri=GOOGLE_REDIRECT_URI.value, ) @@ -572,7 +576,11 @@ def load_oauth_providers(): server_metadata_url=f"{MICROSOFT_CLIENT_LOGIN_BASE_URL.value}/{MICROSOFT_CLIENT_TENANT_ID.value}/v2.0/.well-known/openid-configuration?appid={MICROSOFT_CLIENT_ID.value}", client_kwargs={ "scope": MICROSOFT_OAUTH_SCOPE.value, - "timeout": OAUTH_TIMEOUT.value + **( + {"timeout": int(OAUTH_TIMEOUT.value)} + if OAUTH_TIMEOUT.value + else {} + ), }, redirect_uri=MICROSOFT_REDIRECT_URI.value, ) @@ -596,7 +604,11 @@ def load_oauth_providers(): userinfo_endpoint="https://api.github.com/user", client_kwargs={ "scope": GITHUB_CLIENT_SCOPE.value, - "timeout": OAUTH_TIMEOUT.value + **( + {"timeout": int(OAUTH_TIMEOUT.value)} + if OAUTH_TIMEOUT.value + else {} + ), }, redirect_uri=GITHUB_CLIENT_REDIRECT_URI.value, ) @@ -616,7 +628,9 @@ def load_oauth_providers(): def oidc_oauth_register(client): client_kwargs = { "scope": OAUTH_SCOPES.value, - "timeout": OAUTH_TIMEOUT.value + **( + {"timeout": int(OAUTH_TIMEOUT.value)} if OAUTH_TIMEOUT.value else {} + ), } if (