Skip to content

Pin dompurify to 3.3.1 to resolve Dependabot security alerts#1604

Merged
TalZaccai merged 5 commits intomainfrom
copilot/fix-dependabot-security-issues
Mar 4, 2026
Merged

Pin dompurify to 3.3.1 to resolve Dependabot security alerts#1604
TalZaccai merged 5 commits intomainfrom
copilot/fix-dependabot-security-issues

Conversation

Copy link
Contributor

Copilot AI commented Mar 4, 2026

Post-#1601, the remaining Dependabot alerts stem from dompurify@3.2.6 (transitive via mermaid@docusaurus/theme-mermaid).

Changes

  • website/package.json: Added dompurify: "3.3.1" to npm overrides
  • website/yarn.lock: Bumped dompurify 3.2.6 → 3.3.1

Resolution status

Advisory Affected range Status
GHSA-v8jm-5vwx-cfxm >=3.1.3 <3.2.7 ✅ Fixed by pin to 3.3.1
GHSA-v2wj-7wpq-c8vv >=3.1.3 <=3.3.1 ⚠️ No upstream fix — 3.3.1 is latest; alert will persist until cure53 patches

NuGet packages are clean (dotnet list package --vulnerable reports zero issues).


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits March 4, 2026 21:49
Co-authored-by: TalZaccai <18443527+TalZaccai@users.noreply.github.com>
…utions

Co-authored-by: TalZaccai <18443527+TalZaccai@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix lingering Dependabot security issues Pin dompurify to 3.3.1 to resolve Dependabot security alerts Mar 4, 2026
@TalZaccai TalZaccai marked this pull request as ready for review March 4, 2026 23:20
Copilot AI review requested due to automatic review settings March 4, 2026 23:20
@TalZaccai
Copy link
Contributor

@copilot please remove "resolutions" from package.json

Co-authored-by: TalZaccai <18443527+TalZaccai@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 4, 2026

@copilot please remove "resolutions" from package.json

Removed in 83ccf64. The dompurify pin is now only in npm overrides.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses two remaining Dependabot security alerts in website/ caused by dompurify@3.2.6 (a transitive dependency coming in through mermaid@docusaurus/theme-mermaid). It pins dompurify to 3.3.1 using both npm overrides (for npm package manager) and yarn resolutions (required for Yarn v1, which CI uses via yarn install --frozen-lockfile), and regenerates the yarn.lock to reflect the upgrade.

Changes:

  • website/package.json: Adds "dompurify": "3.3.1" to the overrides section and introduces a new resolutions section with the same pin, ensuring both npm and Yarn v1 resolve the package correctly.
  • website/yarn.lock: Bumps the resolved dompurify entry from 3.2.63.3.1, and includes extensive automatic lock-file reorganizations (entry reordering, alias key consolidation, removal of platform-specific optional packages) produced by the lock file regeneration.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
website/package.json Adds "dompurify": "3.3.1" to npm overrides and a new resolutions block for Yarn v1 compatibility
website/yarn.lock Upgrades dompurify from 3.2.6 to 3.3.1; large set of lock entry reorderings and alias key consolidations from the regeneration run

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@TalZaccai TalZaccai merged commit 5b939b4 into main Mar 4, 2026
12 checks passed
@TalZaccai TalZaccai deleted the copilot/fix-dependabot-security-issues branch March 4, 2026 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants