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

Sanitize user content before creating new idea #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

theonlyrao
Copy link
Owner

@theonlyrao theonlyrao commented Jul 18, 2016

Prevents malicious script attack in the form
<script>alert('Hello');</script> from changing JS on the idea index
page using the ActiveRecord sanitize method.

See http://guides.rubyonrails.org/security.html#injection and http://stackoverflow.com/questions/3009023/how-to-sanitize-sql-fragment-in-rails

@martensonbj, @joshuajhun, @rrgayhart, @chadellison

closes #28

Prevents malicious script attack in the form
`<script>alert('Hello');</script>` from changing JS on the idea index
page using the ActiveRecord `sanitize` method.
@rrgayhart
Copy link

No attacks is good for everyone involved - good catch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

reduce vulnerability to xss attacks
2 participants