Skip to content

Security: Remove sensitive data from log messages - #21

Merged
Macktireh merged 1 commit into
mainfrom
develop
Nov 2, 2025
Merged

Macktireh merged 1 commit into
mainfrom
develop

Conversation

@Macktireh

@Macktireh Macktireh commented Nov 2, 2025 •

Copy link
Copy Markdown
Owner

This PR addresses a critical security vulnerability where sensitive OAuth tokens and credentials could be logged, potentially exposing them in log files or monitoring systems.

@github-actions

github-actions Bot commented Nov 2, 2025 •

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/simplesocialauthlib
   __init__.py00100% 
   abstract.py37392%73, 89, 105
   exceptions.py140100% 
   types.py150100% 
   utils.py211148%26–36
src/simplesocialauthlib/providers
   __init__.py211148%7–8, 14–15, 19–25, 29–30, 34–35
   github.py421369%58–70, 75–82
   google.py471568%71–84, 89–110
TOTAL1975373% 

Tests Skipped Failures Errors Time
11 0 💤 0 ❌ 0 🔥 1.074s ⏱️

@Macktireh
Macktireh merged commit 9038889 into main Nov 2, 2025
9 checks passed
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