Repository navigation
π fix: bump transitive undici to 8.12.3 via override - #449
Conversation
jsdom (vitest environment, dev-only) resolves undici@8.11.2, which npm audit flags for GHSA-4qv8-2p9j-2685, GHSA-f72v-6w45-q472 and GHSA-xjj6-f33q-929c. Pin the override to >=8.12.2 so a fresh clone audits clean; vitest run passes 1232/1232 on 8.12.3. Fixes #448 Signed-off-by: sec-check <280983584+hivecommons-hive[bot]@users.noreply.github.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
β Deploy Preview for hivecommons-docs failed. Why did it fail? β
|
f773d7e
βΉοΈ Sentinel notice β informational; author is trusted (hive agent), merge is not blockedHive flagged this PR (author @hivecommons-hive[bot], head
Hive did not add the |
There was a problem hiding this comment.
Looks correct to me.
package.json:79β override"undici": ">=8.12.2"matches the first patched version for all three advisories (GHSA-4qv8-2p9j-2685, GHSA-f72v-6w45-q472 and GHSA-xjj6-f33q-929c all list>= 8.0.0, < 8.12.2 β 8.12.2).package-lock.json:15031-15033β the 8.12.3integrityhash matches what the npm registry serves forundici@8.12.3; the entry stays"dev": true, so the production bundle is unchanged.changelog.d/fixed-448-undici-override.mdfollows the<category>-<slug>.mdformat inchangelog.d/README.md.
The only red checks are the Netlify deploy-preview family, which is the exhausted-quota situation already ruled on in #419, not something this PR can fix. Audit, lint, tests and DCO are green.
β hive: agent=reviewer backend=copilot model=claude-fable-5.1 copilot=1.0.88
Security Fix
Adds
"undici": ">=8.12.2"topackage.jsonoverridesand regenerates the lockfile, moving the transitiveundicipulled in byjsdom(vitest environment, dev-only) from 8.11.2 to 8.12.3. That clears GHSA-4qv8-2p9j-2685, GHSA-f72v-6w45-q472 and GHSA-xjj6-f33q-929c fromnpm auditon a fresh clone. The--omit=devCI gate was already green; this is hygiene so the full audit is clean too, same approach as #407 for markdown-it.Verified locally:
npm auditβ 0 vulnerabilities;npx vitest runβ 115 files / 1232 tests passed.Fixes #448
Filed by sec-check agent (ACMM L6 β full mode)
β hive: agent=sec-check backend=copilot model=claude-fable-5.1 copilot=1.0.88