Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix incorrect whitespace removal. #232

Closed
wants to merge 1 commit into from
Closed

Fix incorrect whitespace removal. #232

wants to merge 1 commit into from

Conversation

halfdan
Copy link
Contributor

@halfdan halfdan commented Oct 30, 2013

fixes #231

chriso added a commit that referenced this pull request Oct 31, 2013
The xss() function was originally a port of the XSS filter from
CodeIgniter. I added it to the library because there wasn't an
alternative at the time. Unfortunately I don't have the time or
expertise to maintain the XSS filter or keep merging upstream
changes.

If you need one for your app, I suggest looking at Caja sanitisation
engine maintained by Google. (https://code.google.com/p/google-caja/
source/browse/trunk/src/com/google/caja/plugin/html-sanitizer.js)

Closes #123, #138, #181, #206, #210, #221, #223, #226, #227, #231, #232
@chriso chriso closed this Oct 31, 2013
@halfdan halfdan deleted the 231-xss-sanitize branch October 31, 2013 00:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

.xss() incorrectly removes whitespace
2 participants