Cure53
cc @tdeekens who might have more experience on the topic or know who to ask about this :)
@amdw Thanks for looking into this! I'd be happy to change `package.json` if that does the trick on our end. What would have to be changed [here](https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/dompurify/index.d.ts), on this end?
@amdw I have been checking another thread and believe that this might be a first step towards a working fix: https://github.com/cure53/DOMPurify/issues/691#issuecomment-1204059019 I ran the command (slightly modified) and generated the...
Just a heads-up, we updated the README. Maybe this helps solving some of the issues? https://github.com/cure53/DOMPurify#running-dompurify-on-the-server
@amdw thanks a ton for setting up this repo, this is amazing and for the first time allows me to actually debug that problem and understand what is happening :D...
Alright, this should work. Locally, your repo produces an error free build when I do the following changes: * Place type definitions for `purify.cjs.js` and `purify.es.js` in [dist](https://github.com/cure53/DOMPurify/tree/main/dist) folder, [latest...
I think it should be fair to mark this as done :) Please reopen in case anything still doesn't work.
Hi @amdw - yes indeed - we reverted all changes. The fixes worked for some folks (#691, https://github.com/cure53/DOMPurify/issues/705#issuecomment-1214166741) and caused trouble for others (#712, #713), so we decided to roll...
Heya, I am not sure what to do with this ticket. I am not a Node person, don't know what's hot-or-not right now in this realm etc. - this library...
Cool :smile: I would be fine with integrating those if it can be done without affecting current use-cases (my feeling is it won't so all good).