Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Prevent HTML injection through /topic!!!! #344

Merged
merged 1 commit into from Jan 22, 2015
Merged

Conversation

ctian1
Copy link
Contributor

@ctian1 ctian1 commented Jan 22, 2015

Really big security issue here.

Really big security issue here.
@ctian1
Copy link
Contributor Author

ctian1 commented Jan 22, 2015

untested though, beware

@erming
Copy link
Owner

erming commented Jan 22, 2015

Oh, already fixed this issue (here ef041d6) before I saw this. But I might _.escape topic anyway. Merging.

erming added a commit that referenced this pull request Jan 22, 2015
@erming erming merged commit e94341f into erming:master Jan 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants