Skip to content

Security: Add timeouts to all HTTP requests - #22

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 adds timeout configuration to all HTTP requests to prevent indefinite blocking and potential DoS vulnerabilities. Without timeouts, the application could hang indefinitely if the OAuth provider is slow or unresponsive.

@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.py241442%26–39
src/simplesocialauthlib/providers
   __init__.py211148%7–8, 14–15, 19–25, 29–30, 34–35
   github.py431370%59–76, 81–89
   google.py481569%72–86, 91–112
TOTAL2025672% 

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

@Macktireh
Macktireh merged commit da39857 into main Nov 2, 2025
11 of 13 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