Skip to content

More sanitising HTML#990

Merged
edwardcrichton merged 4 commits intodevelopfrom
feature/more-sanitizing
Apr 13, 2026
Merged

More sanitising HTML#990
edwardcrichton merged 4 commits intodevelopfrom
feature/more-sanitizing

Conversation

@jamesrwelch
Copy link
Copy Markdown
Contributor

Use DomPurify to clean html content in various places
Also use the cleanHTML plugin for Jodit to try and stop errors whilst editing HTML

I recommend giving it a good test in the UI to see if you can break it!

@edwardcrichton edwardcrichton force-pushed the feature/more-sanitizing branch from 860a4b7 to 28e012d Compare March 10, 2026 16:10
jamesrwelch and others added 3 commits April 13, 2026 13:55
Also use the `cleanHTML` plugin for Jodit to try and stop errors whilst editing HTML
Remove | safe
Use DOMPurify directly on all this.mergeString = ...
mergeString: string
@edwardcrichton edwardcrichton force-pushed the feature/more-sanitizing branch from 28e012d to 9be3d43 Compare April 13, 2026 12:57
Copy link
Copy Markdown
Contributor

@edwardcrichton edwardcrichton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Team effort; we're both happy with it

@edwardcrichton edwardcrichton merged commit ef7bbe4 into develop Apr 13, 2026
9 checks passed
@edwardcrichton edwardcrichton deleted the feature/more-sanitizing branch April 13, 2026 13:39
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.

3 participants