Repository navigation
Clear open Dependabot security alerts in the lockfile - #45
Merged
Merged
Conversation
Bump toml from 4.3.0 to 5.0.1, the only direct dependency with an open alert (three quadratic-complexity DoS advisories in toml.parse()). The transformer only calls toml.parse() on frontmatter, and 5.x keeps that API, so no code change is needed. Run `npm audit fix` to refresh the transitive copies that the weekly Dependabot version-update PRs never touch, because they only follow package.json entries: brace-expansion 1.1.14 -> 1.1.21 and 5.0.9 -> 5.0.12, js-yaml 3.14.1 -> 3.15.2 (under gray-matter), immutable 5.1.5 -> 5.1.9, picomatch 2.3.1 -> 2.3.2, source-map-js 1.2.1 -> 1.2.2, and fflate 0.7.4 -> 0.7.5. Two alerts remain and need major bumps upstream: katex (via rehype-katex and remark-math) and the fflate copy pinned exactly by satori. Both are build-time only. Verified in a node:22-slim container: npm ci, 78 tests passing, and a full `quartz build` of the 57 content files. Signed-off-by: Geoff Turk <geoff@ic3.dev>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Clears 22 of the 24 open Dependabot alerts. All of them are denial-of-service or ReDoS advisories in build-time tooling, so nothing here reaches the published site, but the alerts were accumulating because the weekly Dependabot version-update PRs only follow package.json entries and never touch nested transitive copies in the lockfile. Dependabot security updates are now enabled on the repo as well, so future transitive alerts will arrive as PRs instead of sitting open.
Changes
tomlfrom 4.3.0 to 5.0.1, the only direct dependency with an open alert (three quadratic-complexity DoS advisories intoml.parse()). The frontmatter transformer only callstoml.parse(), and 5.x keeps that API, so no code change is needed.npm audit fixto refresh the transitive copies: brace-expansion 1.1.14 to 1.1.21 and 5.0.9 to 5.0.12, js-yaml 3.14.1 to 3.15.2 (under gray-matter), immutable 5.1.5 to 5.1.9, picomatch 2.3.1 to 2.3.2, source-map-js 1.2.1 to 1.2.2, and fflate 0.7.4 to 0.7.5.Not addressed
Two alerts remain and need major bumps in upstream packages: katex 0.16 to 0.18 (via rehype-katex and remark-math) and the fflate 0.7.3 copy that satori pins exactly. Both are build-time only and are candidates for dismissal as tolerable risk.
Verification
In a
node:22-slimcontainer: cleannpm ci, a TOML frontmatter smoke test against toml 5.0.1,npm testwith 78 of 78 passing, and a fullquartz buildof all 57 content files. Note thatnpm run checkalready fails onv4because TypeScript 6 rejects the deprecatedmoduleResolution=node10setting in tsconfig.json; that predates this change and is left alone here.