Skip to content

fix: clear supply-chain audit failures - #44

Merged
redanthrax merged 1 commit into
masterfrom
cursor/fix-supply-chain-audit-48d5
Nov 12, 2026
Merged

redanthrax merged 1 commit into
masterfrom
cursor/fix-supply-chain-audit-48d5

Conversation

@redanthrax

Copy link
Copy Markdown
Owner

The Supply chain security workflow on master fails pnpm audit --audit-level=high (7 findings: 2 high, 4 moderate, 1 low). One high is the existing zod ignore. This pins every finding that has a patched release and leaves the audit level unchanged.

Patched

  • follow-redirects 1.16.3 (GHSA-mrwp-f3q8-3g9p, high). Cross-realm sensitiveHeaders arrays. Dev-only via axios.
  • qs 6.16.0 → 6.17.1 (GHSA-6pq8-jhm2-2vcw, moderate). Keeps the earlier qs DoS fixes (GHSA-4mjr-xmp4-gh2g, GHSA-x5fp-wj9c-mxmx).
  • minimatch 10.2.5 → 11.0.7 (GHSA-g3hv-75wg-m249, moderate). 11.0.7 also includes the quadratic-compile fix from that release. 11.0.8 is inside the 24h minimumReleaseAge window (published 2026-11-11).
  • glob 14.0.4 and direct rimraf 6.1.3 → 6.1.4 (GHSA-3g3m-2fjr-6pw4, moderate). rimraf 6.1.4 depends on glob ^14; the override also covers @n8n/node-cli's rimraf 6.0.1, which was still on glob 11. glob 14.0.5 is inside the 24h window (published 2026-11-11).

Unfixable (documented, not silenced by lowering the audit level)

  • deepmerge 4.3.1 (GHSA-gcqx-w48m-w2h4, low). Latest npm release; the advisory has no patched version. Dev-only via @n8n/node-cli → @langchain/community → @browserbasehq/stagehand, and it is not shipped (files is dist/docs/README.md). Ignored in both osv-scanner.toml and auditConfig.ignoreGhsas until 2027-02-12.
  • zod 3.x (GHSA-mrhx-c7xc-m3w5, high) stays ignored. n8n-workflow still requires zod 3, which has no patched release. zod 4 is already pinned to 4.6.7.

Removed the braces ignore (GHSA-vfj7-8cjw-p6xm). GitHub withdrew that advisory on 2026-10-23, and OSV-Scanner reported it as an unused ignore.

Verification

  • pnpm audit --audit-level=high exits 0. Remaining report is 1 low (1 ignored) | 1 high (1 ignored) (deepmerge and zod).
  • OSV-Scanner v2.3.3 (--lockfile=pnpm-lock.yaml) reports no issues after the same two filters.
  • pnpm run audit:supply-chain passed.
  • pnpm install --frozen-lockfile, pnpm run build, and pnpm test (20/20) passed.
  • pnpm run lint is clean aside from @n8n/community-nodes/no-credential-reuse, which does not find package.json when the repo sits directly under / (/workspace). That check passed on the last green master run, where the checkout path is deeper.
Open in Web Open in Cursor 

Pin the high and moderate advisories that were failing pnpm audit and
OSV-Scanner, and drop the withdrawn braces ignore.

- follow-redirects 1.16.3 (GHSA-mrwp-f3q8-3g9p)
- qs 6.16.0 -> 6.17.1 (GHSA-6pq8-jhm2-2vcw, keeps the earlier qs fixes)
- minimatch 10.2.5 -> 11.0.7 (GHSA-g3hv-75wg-m249)
- glob 14.0.4 plus rimraf 6.1.3 -> 6.1.4 (GHSA-3g3m-2fjr-6pw4)
- ignore deepmerge GHSA-gcqx-w48m-w2h4 until 2027-02-12: 4.3.1 is the
  latest release and the advisory has no patched version; dev-only
- remove braces GHSA-vfj7-8cjw-p6xm, withdrawn on 2026-10-23

Co-authored-by: Gary G [Sumerian Software] <redanthrax@users.noreply.github.com>
@redanthrax
redanthrax merged commit 43bfab4 into master Nov 12, 2026
1 check passed
@redanthrax
redanthrax deleted the cursor/fix-supply-chain-audit-48d5 branch November 12, 2026 01:13
redanthrax added a commit that referenced this pull request Nov 12, 2026
Pin the high and moderate advisories that were failing pnpm audit and
OSV-Scanner, and drop the withdrawn braces ignore.

- follow-redirects 1.16.3 (GHSA-mrwp-f3q8-3g9p)
- qs 6.16.0 -> 6.17.1 (GHSA-6pq8-jhm2-2vcw, keeps the earlier qs fixes)
- minimatch 10.2.5 -> 11.0.7 (GHSA-g3hv-75wg-m249)
- glob 14.0.4 plus rimraf 6.1.3 -> 6.1.4 (GHSA-3g3m-2fjr-6pw4)
- ignore deepmerge GHSA-gcqx-w48m-w2h4 until 2027-02-12: 4.3.1 is the
  latest release and the advisory has no patched version; dev-only
- remove braces GHSA-vfj7-8cjw-p6xm, withdrawn on 2026-10-23
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