-
Notifications
You must be signed in to change notification settings - Fork 3
taint vuln #8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
taint vuln #8
Conversation
|
Kudos, SonarCloud Quality Gate passed!
|
|
Kudos, SonarCloud Quality Gate passed!
|
|
Kudos, SonarCloud Quality Gate passed!
|
|
SonarCloud Quality Gate failed.
|
|
Kudos, SonarCloud Quality Gate passed!
|
|
|
Kudos, SonarCloud Quality Gate passed!
|
|
Kudos, SonarCloud Quality Gate passed!
|
|
|
2 similar comments
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
||
| const queryParams = new URLSearchParams(document.location.search); | ||
| const redirectUrl = queryParams.get("url"); | ||
| document.location = redirectUrl; |
Check failure
Code scanning / SonarCloudDev6
DOM updates should not lead to open redirect vulnerabilities High
|
|
|
|
||
| const queryParams = new URLSearchParams(document.location.search); | ||
| const redirectUrl = queryParams.get("url"); | ||
| document.location = redirectUrl; |
Check failure
Code scanning / SonarCloudDev7
DOM updates should not lead to open redirect vulnerabilities High
|
2 similar comments
|
|
|
|
|
|
|
|
🤖 Pull Request summaryAdds URL-based redirect functionality to application initialization. • Extracts "url" query parameter from current page URL and immediately redirects to it Review focus: This introduces a critical security vulnerability - unrestricted redirects can enable phishing attacks. The redirect URL should be validated against an allowlist of safe domains.
|
|
|
|
1 similar comment
|
|
















No description provided.