Comments (7)
Thanks for reporting it @lucamerighi !
Can you provide the full code sample with DOMPurify.sanitize()
parameters?
Did you try 2.7.0 ?
Because isomorphic-dompurify
is just a wrapper around dompurify
, it also makes sense to report the issue to the dompurify issue queue.
from isomorphic-dompurify.
Sure, here's the complete snippet
DOMPurify.sanitize(
`
<html lang="en">
<head>
<title>Sample HTML5 Page</title>
</head>
<body>
<p>Hello</p>
<br>
</body>
</html>`,
{
PARSER_MEDIA_TYPE: "application/xhtml+xml",
}
);
Tried with 2.7.0 and same empty output. Removing the <br>
gives the expected output
from isomorphic-dompurify.
@lucamerighi Thanks. For XHTML, can you use self-closing tags like this <br/>
?
from isomorphic-dompurify.
Yes, in XHTML you can use self closing tags. In fact, you have to convert all void elements to their self-closing variant for it to be valid
from isomorphic-dompurify.
@lucamerighi Well, it's definitely a question/issue for dompurify
developers. I'm afraid I'm not very familiar with internal stuff, so I can't help here.
And I should not suggest switching to the default PARSER_MEDIA_TYPE because you're probably using it for a reason.
from isomorphic-dompurify.
Opened the issue there 🤞 Ye I need it for this reason exactly, parsing HTML into valid XML which between other means means fixing this kind of stuff
from isomorphic-dompurify.
According to the dompurify maintainer, the behavior is by design cure53/DOMPurify#938 (comment) , so I'm closing this issue too.
from isomorphic-dompurify.
Related Issues (20)
- Merge isomorphic-dompurify additions with dompurify? HOT 2
- Add information about the minimum node version HOT 2
- Request body is being removed HOT 3
- High CPU Utilisation by the library HOT 2
- _isomorphicDompurify.default.sanitize is not a function error in jest environment HOT 1
- ESM Support HOT 4
- target blank get added to every url HOT 1
- Build error when using isomorphic-dompurify in angular 15 universal HOT 8
- Requirements for Node.js 14 HOT 1
- Isomorphic Dompurify Remix support ? HOT 2
- Next.js build error: Window is not defined HOT 33
- String is being sanitized HOT 6
- Request for SemVer Adherence in Future Releases HOT 1
- Web Worker Support HOT 3
- usage import issue with vite named export 'sanitize' not found HOT 4
- Cannot find package on server with Nuxt HOT 8
- Can't resolve 'canvas' on next.js serverless app HOT 41
- dompurify.sanitize clears everything except for whats inside of <body> HOT 2
- Bumping to 0.16.0 - ReferenceError: TextEncoder is not defined HOT 24
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from isomorphic-dompurify.