-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
XSS Issues #50
Comments
I found another one. Set the user's first name to <script>alert('x')</script> and you'll get an alert on pretty much every page. |
Group name too. |
Asset tag still has the issue |
I spent some time with this this evening because it was bugging the shit out of me. Weird thing is, the content WAS being escaped on the way out (as you could see from the HTML of course). jquery was actually doing Very Bad Things and sort of re-assembling it. I ended up backing out my alpha_dash changes from earlier, since the laravel validator (stupidly) doesn't have a version of alpha_dash that also allows spaces, and without it, it's too constrictive. I'll be writing a custom validator to reject funky characters at some point soon. Thanks for drawing my attention to this. Was a real titty-twister for much of the night, since I could see the HTML was being escaped. |
I found 3 area with XSS.
1a. Click Delete the alert pops up twice
1b. Add an asset and click on the models drop down the alert shows up
2a. Click Delete, the alert shows up twice
The text was updated successfully, but these errors were encountered: