Skip to content

chore(deps): bump rustls and source-map-js in lockfiles to clear Dependabot alerts - #161

Merged
chinkan merged 2 commits into
mainfrom
chore/lock-alerts
Oct 6, 2026
Merged

chinkan merged 2 commits into
mainfrom
chore/lock-alerts

Conversation

@chinkan

@chinkan chinkan commented Oct 6, 2026

Copy link
Copy Markdown
Owner

Lockfile-only change to clear the two open Dependabot alerts. No manifest or code changes.

Changes

  • Cargo.lock: rustls 0.23.43 -> 0.23.45 (alert: TLS 1.3 handshake messages accepted across encryption level boundaries, fixed in 0.23.45), plus rustls-webpki 0.103.13 -> 0.103.15 pulled by cargo update -p rustls --precise 0.23.45.
    • rustls is not compiled into RustFox: cargo tree -i rustls --target all prints nothing (TLS is native-tls since chore(deps)(deps): bump reqwest from 0.12.28 to 0.13.4 #134). The entry stays in the lockfile through reqwest 0.13's optional hyper-rustls, so removing it by hand would be re-added by Cargo. Bumping it is the stable fix.
  • web/package-lock.json: source-map-js 1.2.1 -> 1.2.2 (alert: event-loop DoS via indexed source-map section offsets). Edited the single entry (version, resolved, integrity from npm view source-map-js@1.2.2) to avoid unrelated churn from a different npm version.

Verification

  • cargo clippy --locked --all-targets -- -D warnings clean, cargo test --locked: 921 passed.
  • web/: npm ci installs source-map-js 1.2.2, 0 vulnerabilities; npm run verify (typecheck, build, 39 tests) passes on Node 22.

…ndabot alerts

Cargo.lock: rustls 0.23.43 -> 0.23.45 (and rustls-webpki 0.103.13 -> 0.103.15).
The entry is not built (cargo tree -i rustls --target all is empty) but stays
in the lockfile through reqwest 0.13's optional hyper-rustls, so it is bumped
rather than removed.
web/package-lock.json: source-map-js 1.2.1 -> 1.2.2.

chinkan commented Oct 6, 2026

Copy link
Copy Markdown
Owner Author

TL ACCEPT at 3464dee (comment — same-account Approve blocked).

Lock-only: rustls 0.23.45 + rustls-webpki 0.103.15 (cannot drop rustls; optional hyper-rustls keeps the entry), source-map-js 1.2.2 single-entry bump. Merge after #160 (done) + QA GO (CI + npm run verify) + Product GO. Product AC: Dependabot alerts for those two clear on main after merge. AI findings 402 non-block. No tag.

chinkan commented Oct 6, 2026

Copy link
Copy Markdown
Owner Author

TL ACCEPT reaffirmed at 7c2eb9f (Update branch merge onto 6b2cf0d). Diff vs main still lockfile-only (+7/−7). Merge after QA GO + Product GO + Analyze (rust) green. No tag.

@chinkan
chinkan merged commit 5a94889 into main Oct 6, 2026
13 of 14 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.

1 participant