Skip to content

Commit 49de9e2

Browse files
dpgasparclaude
andauthored
fix(security): remove OAuth token values from debug log statements (#2440)
OAuth tokens and full OAuth responses were being logged at debug level, which could expose bearer credentials in log files (CWE-532). [sc-100058] Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 5099b21 commit 49de9e2

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

flask_appbuilder/security/manager.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ def _oauth_tokengetter(token=None):
134134
from session cookie.
135135
"""
136136
token = session.get("oauth")
137-
log.debug("Token Get: %s", token)
137+
log.debug("Token Get: retrieved OAuth token from session")
138138
return token
139139

140140

flask_appbuilder/security/views.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -698,7 +698,7 @@ def oauth_authorized(self, provider: str) -> WerkzeugResponse:
698698
if resp is None:
699699
flash("You denied the request to sign in.", "warning")
700700
return redirect(self.appbuilder.get_url_for_login)
701-
log.debug("OAUTH Authorized resp: %s", resp)
701+
log.debug("OAUTH Authorized resp received for provider: %s", provider)
702702
# Retrieves specific user info from the provider
703703
try:
704704
self.appbuilder.sm.set_oauth_session(provider, resp)

0 commit comments

Comments
 (0)