-
Notifications
You must be signed in to change notification settings - Fork 10
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
Do something to prevent XSS #54
Comments
that's meant as a band-aid until a proper fix can be applied to a site |
@jeroldhaas Ok good, info. I will definitely do more research before I implement something. |
Perhaps I can assist. Can you provide more info on this bug? Is text entered into textboxes getting |
Basic Idea is that I know there is a lot of things about web dev that I don't know, and managing and preventing XSS is one of them, so this is a todo for me to research and implement a fix. I did test it and put a script tag in an input box, and after saving it, it did evaluate the script tag on rendering the 'view' page. |
These might be a good starting point - there are some more but their compatibility with Suave might be suspect: |
You can enter <script>alert('hello')</script> into the text box and it will run the js when loading the view page.
The text was updated successfully, but these errors were encountered: