Update dependency svgo to v4.0.2 [SECURITY]

[renovate]

This PR contains the following updates:

Package	Change	Age	Confidence
svgo (source)	4.0.1 → 4.0.2

---

SVGO removeScripts plugin leaves some executable scripts intact

GHSA-2p49-hgcm-8545

More information

Details

Summary

SVGO's removeScripts plugin (disabled by default) removes scripts from the SVG, however executable scripts were left intact in some cases. If a consumer relied on this plugin for sanitization and served them to users, these SVGs could open up doors to XSS.

Details

SVGO has a plugin for removing scripts from an SVG, which removes:

• <script> elements
• JavaScript URIs (v4 and v3 only)
• on… event handlers (v4 and v3 only)

While SVGO is not a sanitization library, SVGO continues to maintain the plugin for those already using it for this purpose.

However, there were two problems:

• SVGO did not check namespaced/prefixed script elements, for example if one declared an explicit prefix for the SVG namespace ( <svg:script>) instead of using the default namespace ( <script>), the <svg:script> tag would be left intact.
• SVGO case sensitively matched JavaScript URIs, but it should've been case-insensitive.

Proof of Concept

import { optimize } from 'svgo';

/** Presume that this string was obtained in some other way, such as network. */
const original = `
  <svg xmlns="http://www.w3.org/2000/svg" xmlns:svg="http://www.w3.org/2000/svg" xmlns:uwu="http://www.w3.org/1999/xlink" viewBox="0 0 100 100" version="1.1">
    <a uwu:href="JavaScript:(() =&gt; { alert(document.cookie) })();"><text y="30">uwu</text></a>
    <svg:script>
      alert(document.cookie);
    </svg:script>
  </svg>
`;

optimize(original, {
  plugins: ['removeScripts']
});
// Did not remove <svg:script> or uwu:href="JavaScript:…—still executed by browsers.

Impact

If you run SVGO on untrusted input (e.g., user uploads to a web application) and you depended on removeScripts, then some scripts may still be present. If that SVG was then opened directly by another user on the same domain, it could invoke scripts that could read local storage or cookies.

This may affect you if you have enabled one of the following:

SVGO Version	Plugin Name
v4	removeScripts
v3	removeScriptElement
v2	removeScriptElement
v1	removeScriptElement

It's unlikely to impact users who just use SVGO locally on their own SVGs or in build pipelines.

Patches

>= 3.0.0, <= 4.0.1

SVGO patched v4.0.2 and v3.3.4. Just upgrade the dependency using your preferred package manager! For example:

yarn up svgo

##### or if SVGO is a nested dependency
yarn up -R svgo

The proposed fix is to improve our namespace-aware handling to explicitly act on the default namespace, SVG namespace, and XHTML namespace only. This handles all scripts that are executed by browsers, but will leave intact custom prefixes that happen to have an element called <*:script> which clients shouldn't treat as executable.

>= 2.0.0, <= 2.8.2

SVGO patched v2.8.3, however SVGO v2 explicitly only implements and documents that it will remove <script> elements and nothing more. It has the namespace aware handling for tags like <svg:script> or <xhtml:script>, but has not been updated to remove JavaScript URIs or event handlers like >= v3. If this is something you need, please upgrade to v4 or v3, or reach for one of the documented workarounds at the end.

>= 1.0.0, <= 1.3.2

SVGO v1 has been deprecated for a while now and won't be patched. Please upgrade to a more recent version! If something is preventing you from doing so, please reach out! We're happy to expand our migration guides or support you if you're having trouble.

Workarounds

If your motivation for enabling the plugin is SVG sanitization, consider reaching for a dedicated SVG sanitization tool and invoke it before passing the SVG to SVGO.

Severity

• CVSS Score: 8.2 / 10 (High)
• Vector String: CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:H/I:L/A:N

References

• https://github.com/svg/svgo/security/advisories/GHSA-2p49-hgcm-8545
• https://github.com/svg/svgo/commit/628e3bc7336625a30365d0a9b60185307d852466
• https://github.com/svg/svgo/commit/72a23886b4698b27624b936f3a15a80afd36d75f
• https://github.com/svg/svgo/commit/f529cfccc6c154d6f6eabe276ec637a8c5db6763
• https://github.com/svg/svgo/releases/tag/v2.8.3
• https://github.com/svg/svgo/releases/tag/v3.3.4
• https://github.com/svg/svgo/releases/tag/v4.0.2
• https://github.com/advisories/GHSA-2p49-hgcm-8545

This data is provided by the GitHub Advisory Database (CC-BY 4.0).

---

Release Notes

svg/svgo (svgo)

v4.0.2

Compare Source

What's Changed

Security

• removeScripts, remove JavaScript URIs case-insensitively and make <script> handling namespace aware. By @​SethFalco

Bug Fixes

• convertColors, skip case-normalization for CSS custom properties. By @​SethFalco in #​2213
• convertPathData, improve handling of consecutive t commands. By @​KTibow in #​2156
• convertPathData, safer handling of paths. By @​KTibow in #​2158
• Improve parsing of negative (or +) arc radii. By @​KTibow in #​2205
• When converting to a data URI, only encode it as a data URI once. By @​johnkenny54 in #​2053

Performance

• mergePaths, speed up child node removal. By @​mozzie in #​2216

Other Changes

• removeAttributesBySelector, log warning and noop if missing parameters. By @​KTibow in #​2215

---

Configuration

📅 Schedule: (UTC)

• Branch creation
  • At any time (no schedule defined)
• Automerge
  • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.

---

• If you want to rebase/retry this PR, check this box

---

This PR was generated by Mend Renovate. View the repository job log.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.64%. Comparing base (172ce56) to head (164be59).

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1574   +/-   ##
=======================================
  Coverage   63.64%   63.64%           
=======================================
  Files          68       68           
  Lines        6206     6206           
  Branches     1378     1378           
=======================================
  Hits         3950     3950           
  Misses       2217     2217           
  Partials       39       39           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.