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

Html escaping #175

Merged
merged 1 commit into from Jun 3, 2012
Merged

Conversation

everett-sochowski
Copy link
Contributor

With the update to the underscore library we can now make use of html escaping in the underscore templates. This fixes the case where e.g. you could enter script tags as a todo and it have it execute, or enter italic tags and get italics displayed.

@sindresorhus
Copy link
Member

The Backbone example in master is old. Can you apply it to the one in the Backbone branch instead?

@everett-sochowski
Copy link
Contributor Author

Hi, the html_escaping branch I created was actually based off of the backbone branch. I'm still quite new to the whole github thing but as far as I can tell my pull request is asking to merge back into the backbone branch as well. If I'm mistaken about something, please let me know. Thanks!

@sindresorhus
Copy link
Member

Oh, sorry, I must have assumed it weren't or something.

sindresorhus added a commit that referenced this pull request Jun 3, 2012
Html escaping of Backbone underscore template
@sindresorhus sindresorhus merged commit ec1c9a3 into tastejs:backbone Jun 3, 2012
@everett-sochowski everett-sochowski mentioned this pull request Jun 3, 2012
gustaff-weldon pushed a commit to gustaff-weldon/todomvc that referenced this pull request Dec 23, 2013
Html escaping of Backbone underscore template
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants