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 vulnerable #41

Open
stuartpb opened this issue Apr 15, 2015 · 1 comment
Open

XSS vulnerable #41

stuartpb opened this issue Apr 15, 2015 · 1 comment

Comments

@stuartpb
Copy link

This module creates content for an inline <script> tag. However, if anything in the content of that script tag includes </script>, HTML5's parsing rules will prematurely end the tag there, allowing for XSS injection.

This module should modify any strings in the returned JSON (for keys or values) that include </script> to be written as </"+"script>.

@freewil
Copy link

freewil commented Apr 15, 2015

I can't confirm whether this allows for an XSS injection vulnerability, but I do believe #34 fixes your concern.

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