Skip to content

Stop logging OAuth secrets#148

Open
zerox80 wants to merge 1 commit intoopencloud-eu:mainfrom
zerox80:security/redact-auth-logs
Open

Stop logging OAuth secrets#148
zerox80 wants to merge 1 commit intoopencloud-eu:mainfrom
zerox80:security/redact-auth-logs

Conversation

@zerox80
Copy link
Copy Markdown
Contributor

@zerox80 zerox80 commented May 8, 2026

What changed

  • Removed OAuth code, token and state values from auth logs.
  • Redacted sensitive request and response fields in HTTP debug logs.
  • Made token response toString output safe, so accidental logs do not dump tokens.

Why

Some debug/support logs could include OAuth credentials. This keeps the useful flow logs, but drops the secret values.

Checks

  • git diff --check
  • Targeted rg scans for the old token/code log messages
  • Gradle tests not run locally because JAVA_HOME is not set and java is not on PATH

@zerox80 zerox80 marked this pull request as ready for review May 8, 2026 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant