Skip to content

fix: bump astro 5.18.0 + rollup 4.59.0, clear high-severity vulns - #6

Merged
brickfrog merged 1 commit into
masterfrom
fix/astro-bump-5.18
Mar 2, 2026
Merged

fix: bump astro 5.18.0 + rollup 4.59.0, clear high-severity vulns#6
brickfrog merged 1 commit into
masterfrom
fix/astro-bump-5.18

Conversation

@0x4m4d3u5

Copy link
Copy Markdown
Collaborator

Summary

Before: 10 vulnerabilities (4 high, 3 moderate, 3 low)
After: 3 moderate (ajv + lodash in @astrojs/check dev chain, lodash-es in mermaid — no upstream fix available)

Note: bun audit still shows rollup as flagged but this is a false positive — lockfile confirms rollup@4.59.0 which is the patched version; advisory range is <4.59.0.

Test plan

  • bun run verify passes (format, lint, markdown lint, 173 unit tests, astro type check — all clean)
  • Pre-commit hook ran full verify suite before commit

astro 5.16.0 → 5.18.0 resolves:
- h3 high (request smuggling) via unstorage update
- devalue high x2 (DoS via memory/CPU exhaustion)
- diff low (DoS in parsePatch)

rollup pinned to ^4.59.0 as direct dep to resolve:
- GHSA-mw96-cpmx-2vgc (arbitrary file write via path traversal)

Also corrects @astrojs/rss constraint to ^4.0.15 (lockfile was already
updated in #5, package.json constraint lagged behind).

Remaining: 3 moderate vulns in @astrojs/check language-server chain
(ajv, lodash) and mermaid (lodash-es) — no upstream fix available.
All 173 tests pass, astro check clean.
@brickfrog
brickfrog merged commit 634e692 into master Mar 2, 2026
2 checks passed
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.

2 participants