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.

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

Closed
nlipke opened this issue Nov 26, 2013 · 4 comments
Closed

XSS Issues #50

nlipke opened this issue Nov 26, 2013 · 4 comments

Comments

@nlipke
Copy link

nlipke commented Nov 26, 2013

I found 3 area with XSS.

  1. Create an Model with a name of <script>alert('a')</script>
    1a. Click Delete the alert pops up twice
    1b. Add an asset and click on the models drop down the alert shows up
  2. Create an Asset with a name of <script>alert(1)</script>
    2a. Click Delete, the alert shows up twice
@nlipke
Copy link
Author

nlipke commented Nov 26, 2013

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.

@nlipke
Copy link
Author

nlipke commented Nov 26, 2013

Group name too.

@snipe snipe closed this as completed in 4d1dd1c Nov 26, 2013
@nlipke
Copy link
Author

nlipke commented Nov 26, 2013

Asset tag still has the issue

snipe added a commit that referenced this issue Nov 26, 2013
@snipe
Copy link
Owner

snipe commented Nov 26, 2013

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.

This was referenced Jul 19, 2016
@Bjufen Bjufen mentioned this issue Feb 9, 2023
2 tasks
This was referenced Aug 22, 2023
@manhvt20 manhvt20 mentioned this issue Mar 18, 2024
2 tasks
@uglycrab uglycrab mentioned this issue Mar 27, 2024
2 tasks
@jayavman jayavman mentioned this issue Jul 8, 2024
2 tasks
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

No branches or pull requests

2 participants