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

fix: apply DOMPurify.santize on ReactMarkdownOrHtml #2483

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

jessicamcinchak
Copy link
Member

@jessicamcinchak jessicamcinchak commented Nov 27, 2023

Replaces #2478 by addressing issue at first point of user interaction in the client. Still requires #2479 to fix root issue.

Copy link

github-actions bot commented Nov 27, 2023

Removed vultr server and associated DNS entries

@jessicamcinchak jessicamcinchak changed the title fix: apply DOMPurify.santize on ReactMarkdownOrHtml <HTMLRoot /> fix: apply DOMPurify.santize on ReactMarkdownOrHtml Nov 27, 2023
@jessicamcinchak jessicamcinchak marked this pull request as ready for review November 27, 2023 12:31
@jessicamcinchak jessicamcinchak requested a review from a team November 27, 2023 12:31
@DafyddLlyr
Copy link
Contributor

Perfect - works as expected!

@jessicamcinchak jessicamcinchak merged commit 8f42ff4 into main Nov 27, 2023
12 checks passed
@jessicamcinchak jessicamcinchak deleted the jess/dompurify-react-markdown-html branch November 27, 2023 13:30
@jessicamcinchak
Copy link
Member Author

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.

2 participants