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

Fewer link_to warnings #1093

Merged
merged 7 commits into from Sep 7, 2017

Conversation

Projects
None yet
1 participant
@presidentbeef
Owner

presidentbeef commented Sep 5, 2017

Only warn on direct user input and model attributes that look like URLs.

@presidentbeef presidentbeef merged commit 19f0923 into master Sep 7, 2017

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
ci/circleci Your tests passed on CircleCI!
Details

@presidentbeef presidentbeef deleted the less_link_to branch Sep 7, 2017

@presidentbeef presidentbeef referenced this pull request Sep 13, 2017

Closed

XSS False Positive #1081

Repository owner locked and limited conversation to collaborators Oct 18, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.