Skip to content

fix(deps): address smol-toml DoS vulnerability (GHSA-v3rj-xjv7-4jmq)#783

Merged
BYK merged 2 commits intomasterfrom
byk/fix/smol-toml-security
Mar 25, 2026
Merged

fix(deps): address smol-toml DoS vulnerability (GHSA-v3rj-xjv7-4jmq)#783
BYK merged 2 commits intomasterfrom
byk/fix/smol-toml-security

Conversation

@BYK
Copy link
Copy Markdown
Member

@BYK BYK commented Mar 25, 2026

Fixes Dependabot alert #132.

smol-toml@1.6.0 (transitive dependency of astro and @astrojs/markdown-remark in the docs sub-project) is vulnerable to a medium-severity DoS via TOML documents containing thousands of consecutive commented lines, causing a stack overflow from uncontrolled recursion.

Fix: Add a pnpm override in docs/package.json to force smol-toml@^1.6.1, which replaces the recursive comment parser with an iterative approach.

No changes to the main Craft CLI — this only affects the docs site build dependencies.

Add pnpm override in docs/package.json to force smol-toml@^1.6.1,
fixing a medium-severity DoS via deeply nested commented TOML lines.
The vulnerable version (1.6.0) was a transitive dependency of astro
and @astrojs/markdown-remark in the docs sub-project.
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 25, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-03-25 23:37 UTC

@BYK BYK marked this pull request as ready for review March 25, 2026 23:33
@BYK BYK merged commit 7223a48 into master Mar 25, 2026
19 checks passed
@BYK BYK deleted the byk/fix/smol-toml-security branch March 25, 2026 23:36
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.

1 participant