Repository navigation
fix(deps): pin undici content-encoding and EventSource patch - #42
Conversation
Pins the discord.js transitive undici to exactly 6.29.1, addressing GHSA-xjj6-f33q-929c (high) and GHSA-4qv8-2p9j-2685 (moderate). Claude-Session: https://claude.ai/code/session_01NrbT7UzDQzESHKsvnJAgps
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Pin undici override package.json |
The undici override changes from ^6.28.0 to the exact version 6.29.1. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Possibly related PRs
- tarkovtracker-org/TrackerBot#38: Adds the
audit:prodgate and addresses production dependency advisories, including theundiciadvisory covered by this version pin.
Suggested reviewers: nivmizz7
Merge Risk: 
Minimal · up to a5bc9
-
No merge-blocking issue was found in this dependency-only change.
Pre-merge checks: 
5
-
Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
Advisory chain
Pins the
discord.js→undicitransitive to exactly6.29.1(was6.28.1via the^6.28.0override), fixing:Vetting
6.29.1published 2026-10-15 (past the 7-day cooldown), same GitHub Actions trusted publisher with SLSA provenance as6.28.1, no lifecycle-script changes.undicientry.Verification
npm ci --ignore-scripts && npm test: 11/11 pass.npm audit --omit=devno longer reportsundici/discord.js/@discordjs/rest. Remaining standalone advisories (body-parser,follow-redirects,ip-address) are fixed in sibling PRs and combined in fix(web): share report limits and surface submission errors #38, whoseaudit:prodCI gate is the regression check.Integration
Merged into #38 alongside #39/#40 so #38's merge commit lands every advisory fix together.
https://claude.ai/code/session_01NrbT7UzDQzESHKsvnJAgps
Summary by cubic
Pins the
undicitransitive dependency (viadiscord.js) to exactly6.29.1, fixing GHSA-xjj6-f33q-929c (unhandled error on excessive content-encodings) and GHSA-4qv8-2p9j-2685 (unbounded memory in the EventSource parser).Verification
npm ci --ignore-scripts && npm testpasses 11/11.npm audit --omit=devno longer reportsundici,discord.js, or@discordjs/rest.Written for commit a5bc9f5. Summary will update on new commits.