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

Add sanitization for potential XSS vulnerability #71

Closed
justinmahar opened this issue Apr 2, 2024 · 0 comments
Closed

Add sanitization for potential XSS vulnerability #71

justinmahar opened this issue Apr 2, 2024 · 0 comments
Assignees
@justinmahar

Comments

@justinmahar
Copy link
Owner

justinmahar commented Apr 2, 2024 •
edited
Loading

Brought to my attention by @gtsp233, this library would benefit from built-in sanitization of post URLs.

PR addressing sanitization:

The following package seems like a good fit for all-purpose sanitization, should we want to take this a step further. For now, XSS vuln protection covers the placeholder only. DOMpurify: https://www.npmjs.com/package/dompurify
@justinmahar justinmahar self-assigned this Apr 2, 2024
@justinmahar justinmahar changed the title Add DOMPurify sanitization Add DOMPurify sanitization for potential XSS vulnerability Apr 2, 2024
@justinmahar justinmahar changed the title Add DOMPurify sanitization for potential XSS vulnerability Add sanitization for potential XSS vulnerability Apr 3, 2024
@justinmahar justinmahar mentioned this issue Apr 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@justinmahar justinmahar

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@justinmahar