refac
This commit is contained in:
@@ -135,6 +135,7 @@ class OAuthSessionTable:
|
|||||||
db.refresh(result)
|
db.refresh(result)
|
||||||
|
|
||||||
if result:
|
if result:
|
||||||
|
db.expunge(result) # Detach so dict swap is never flushed
|
||||||
result.token = token # Return decrypted token
|
result.token = token # Return decrypted token
|
||||||
return OAuthSessionModel.model_validate(result)
|
return OAuthSessionModel.model_validate(result)
|
||||||
else:
|
else:
|
||||||
|
|||||||
@@ -1706,17 +1706,22 @@ class OAuthManager:
|
|||||||
db=db,
|
db=db,
|
||||||
)
|
)
|
||||||
|
|
||||||
response.set_cookie(
|
if session:
|
||||||
key="oauth_session_id",
|
response.set_cookie(
|
||||||
value=session.id,
|
key="oauth_session_id",
|
||||||
httponly=True,
|
value=session.id,
|
||||||
samesite=WEBUI_AUTH_COOKIE_SAME_SITE,
|
httponly=True,
|
||||||
secure=WEBUI_AUTH_COOKIE_SECURE,
|
samesite=WEBUI_AUTH_COOKIE_SAME_SITE,
|
||||||
)
|
secure=WEBUI_AUTH_COOKIE_SECURE,
|
||||||
|
)
|
||||||
|
|
||||||
log.info(
|
log.info(
|
||||||
f"Stored OAuth session server-side for user {user.id}, provider {provider}"
|
f"Stored OAuth session server-side for user {user.id}, provider {provider}"
|
||||||
)
|
)
|
||||||
|
else:
|
||||||
|
log.warning(
|
||||||
|
f"Failed to create OAuth session for user {user.id}, provider {provider}"
|
||||||
|
)
|
||||||
except Exception as e:
|
except Exception as e:
|
||||||
log.error(f"Failed to store OAuth session server-side: {e}")
|
log.error(f"Failed to store OAuth session server-side: {e}")
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user