videojs / mpd-parser

https://videojs.github.io/mpd-parser/
Other
79 stars 54 forks source link

Security vulnerability in xmldom #143

Closed andrewspinks closed 3 years ago

andrewspinks commented 3 years ago

npm is reporting a security vulnerability in a xmldom which is causing npm audit to fail for users of video.js and mpd-parser. https://www.npmjs.com/advisories/1769

security-vuln

Unfortunately, a fix has not yet been published, so I cannot open a PR to address this yet. https://github.com/xmldom/xmldom/issues/271

gkatsev commented 3 years ago

It's worth noting that xmldom isn't used by mpd-parser in the built files that run on a webpage. It's only used by mpd-parser if you use it via the CLI or via node.js directly.

dj-documentation commented 3 years ago

videojs/video.js#7389

dj-documentation commented 3 years ago

Leaving video.js and mpd-parser unpatched trains users to ignore npm audit, yarn audit, and GitHub warnings.

image

dj-documentation commented 3 years ago

xmldom is now @xmldom/xmldom

https://github.com/xmldom/xmldom/issues/271