Skip to content
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.

Sign up for GitHub

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

Jump to bottom

XSS vulnerability in login redirect param #184

Closed
SocalNick opened this issue Jan 16, 2015 · 1 comment
Closed

XSS vulnerability in login redirect param #184

SocalNick opened this issue Jan 16, 2015 · 1 comment

Comments

@SocalNick
Copy link
Owner

Security advisory: XSS vulnerability in login redirect param

ScnSocialAuth version 1.15.2 has been released and includes a security for this vulnerability. Fix has been applied in 4a00966

Affected versions

All versions below 1.15.2 are affected. dev-master is fixed starting from 4a00966

Exploits

Because of missing escaping of the URL param redirect a XSS attack is possible.
For example: Setting the redirect param to "><a%20href="http://github.com">GitHub.com</a><inpu%20type="hidden"%20" would result in a link added to the login page.

Resolution

If you are using any version of ScnSocialAuth below 1.15.2 please upgrade immediately by running composer update.

Credits

The vulnerability was reported by @Danielss89
The original vulnerability in ZfcUser was discovered and fixed by @GyunerZeki (See ZF-Commons/ZfcUser#550)
@SocalNick
Copy link
Owner Author

Fixed in https://github.com/SocalNick/ScnSocialAuth/releases/tag/1.15.2

SocalNick added a commit to SocalNick/ScnSocialAuth-Sandbox that referenced this issue Jan 16, 2015
@SocalNick
Update ScnSocialAuth to >= 1.15.2
478f660 
- Also updates ZfcUser to 1.2.2
- See security issue:
SocalNick/ScnSocialAuth#184
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SocalNick