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.

Sign up for GitHub

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

Jump to bottom

Sanitize URL in preview #119

Merged
merged 3 commits into from Sep 13, 2013
Merged

Sanitize URL in preview #119

merged 3 commits into from Sep 13, 2013

Conversation

jzeta
Copy link
Contributor

@jzeta jzeta commented Sep 10, 2013

Make sure oneboxes don't allow XSS injection that would impact security

@ghost ghost assigned vykster Sep 10, 2013
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 41b2228 on sanitize-url-for-preview into 3f20142 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e742cda on sanitize-url-for-preview into 3f20142 on master.

vykster added a commit that referenced this pull request Sep 13, 2013
@vykster
Merge pull request #119 from dysania/sanitize-url-for-preview
e3450d2 
Sanitize URL in preview
@vykster vykster merged commit e3450d2 into master Sep 13, 2013
@vykster vykster deleted the sanitize-url-for-preview branch September 13, 2013 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
1.0.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jzeta @coveralls @vykster