updating html sanitization method to dompurify

sourcegraph / openctx

See contextual info about code from your dev tools, in your editor, code review, and anywhere else you read code.

https://openctx.org

Apache License 2.0

149 stars 17 forks source link

updating html sanitization method to dompurify #172

Open mohammadualam opened 3 months ago

mohammadualam commented 3 months ago

Our standard way of sanitizing HTML is using dompurify. we want to stay consistent across our code for that and this update changes over the sanitization to dompurify.

keegancsmith commented 3 months ago

Your CI failures are super weird, its almost like the github workflow's code changed for npm and is now broken. I think to sort it out we should try bumping the versions of the workflows used.

keegancsmith commented 3 months ago

Gonna see if this fixes the problem https://github.com/sourcegraph/openctx/pull/174