Skip to content

checker: detect user data from python request passed into method calls causing injection vulnerabilities#170

Merged
sourya-deepsource merged 5 commits intoDeepSourceCorp:masterfrom
MashyBasker:checkers/django-tainted-data-injection
Mar 21, 2025
Merged

checker: detect user data from python request passed into method calls causing injection vulnerabilities#170
sourya-deepsource merged 5 commits intoDeepSourceCorp:masterfrom
MashyBasker:checkers/django-tainted-data-injection

Conversation

@MashyBasker
Copy link
Contributor

@MashyBasker MashyBasker commented Mar 19, 2025

Purpose

This PR adds a checkers to detect

  • SSRF attacks due to user data received from request calls being passed into server side get requests.
  • SQL injection attacks due to user data from request calls being passed into RawSQL, cursor.execute and object.raw() methods

@vercel
Copy link

vercel bot commented Mar 19, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
globstar ⬜️ Ignored (Inspect) Visit Preview Mar 21, 2025 3:24pm

@MashyBasker MashyBasker changed the title checker: detect user data passed into server side requests resulting in potential SSRF attacks checker: detect user data from python request passed into method calls causing injection vulnerabilities Mar 19, 2025
…in potential SSRF attacks

Signed-off-by: Maharshi Basu <basumaharshi10@gmail.com>
Signed-off-by: Maharshi Basu <basumaharshi10@gmail.com>
…thods

Signed-off-by: Maharshi Basu <basumaharshi10@gmail.com>
Signed-off-by: Maharshi Basu <basumaharshi10@gmail.com>
Signed-off-by: Maharshi Basu <basumaharshi10@gmail.com>
@MashyBasker MashyBasker force-pushed the checkers/django-tainted-data-injection branch from e1c02af to 7bd9cba Compare March 21, 2025 15:24
@sourya-deepsource sourya-deepsource merged commit ce419ba into DeepSourceCorp:master Mar 21, 2025
3 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.

2 participants