Open-redirect via login ?next=-parameter #102
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
jesse-a/OpenCRM#102
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Severity: MEDIUM (v0.9)
De login-redirect-flow gebruikte de
next-query-parameter als doel zonder validatie tegen protocol-relative URLs (//evil.com) of absolute URLs. Een phishing-link kon zo na succesvolle login de gebruiker naar een externe site sturen.Fix:
safeNextPath()weigert URLs die niet single-slash same-origin zijn, ASCII-control chars en URL-encoded slashes.Opgelost in commit
9fe44cf.