@diosmosis opened this Pull Request on August 11th 2019 Member
@diosmosis commented on August 16th 2019 Member

Updated.

@tsteur commented on August 16th 2019 Member

Haven't tested it but looks good @diosmosis can you just check it escapes things before merging?

@diosmosis commented on August 17th 2019 Member

Verified the notification escapes the content before displaying; adding <script>alert(1);</script> did not create an alert.

This Pull Request was closed on August 17th 2019
Powered by GitHub Issue Mirror