Skip to content

Add OpenJS CNA credentials and Worker client to reserve CVEs - #1232

Merged
aduh95 merged 3 commits into
nodejs:mainfrom
UlisesGascon:feat/openjs-cna-reserve
Nov 5, 2026
Merged

aduh95 merged 3 commits into
nodejs:mainfrom
UlisesGascon:feat/openjs-cna-reserve

Conversation

@UlisesGascon

@UlisesGascon UlisesGascon commented Oct 28, 2026 •

Copy link
Copy Markdown
Member

@codecov

codecov Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 70.44025% with 94 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.52%. Comparing base (8f02c22) to head (bdfd15d).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
lib/update_security_release.js 66.01% 70 Missing ⚠️
lib/auth.js 17.24% 24 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1232      +/-   ##
==========================================
+ Coverage   65.07%   66.52%   +1.45%     
==========================================
  Files          66       69       +3     
  Lines       13205    14103     +898     
==========================================
+ Hits         8593     9382     +789     
- Misses       4612     4721     +109     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

UlisesGascon added a commit to UlisesGascon/node that referenced this pull request Oct 28, 2026
When `cve_source` is set to `openjs-cna` in `.ncurc`, `git node security
--request-cve` reserves CVE ids through the OpenJS Foundation CNA instead
of HackerOne. Publication to MITRE is then handled by the `Publish CVEs`
workflow in the nodejs-private/security-release repository, which runs
when the security release pull request is merged and holds until every
patched version is on nodejs.org/dist.

Update the security release runbook accordingly:

  * Note the OpenJS CNA option under 'Requesting CVEs', including the
    resume behaviour on supervised buckets.
  * Mark 'Request publication of H1 CVE requests' in cleanup as a no-op
    under the OpenJS CNA path.
  * Add a Post-Release step describing the automatic publication, the
    nodejs.org gate, the failure issue and the delay before cve.org.
  * Scope the HackerOne form under 'Updating CVEs' to HackerOne-issued
    ids and point OpenJS-issued ids at the CNA coordinators.

Refs: openjs-foundation/security-wg#337
Refs: nodejs/node-core-utils#1232
Refs: nodejs-private/security-release#87
Assisted-by: Claude
Signed-off-by: Ulises Gascon <ulisesgascongonzalez@gmail.com>
@UlisesGascon
UlisesGascon marked this pull request as ready for review November 5, 2026 10:23
@UlisesGascon
UlisesGascon force-pushed the feat/openjs-cna-reserve branch from a0a06ce to bdfd15d Compare November 5, 2026 10:25
@UlisesGascon

Copy link
Copy Markdown
Member Author

I solved the commit linting issues @aduh95 👍

@aduh95 aduh95 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A bit rubber-stampy, but LGTM

@aduh95
aduh95 merged commit b2b3864 into nodejs:main Nov 5, 2026
11 checks passed
UlisesGascon added a commit to UlisesGascon/node that referenced this pull request Nov 6, 2026
When `cve_source` is set to `openjs-cna` in `.ncurc`,
`git node security --request-cve` reserves CVE ids through the OpenJS
Foundation CNA instead of HackerOne. Publication to MITRE is then
handled by the `Publish CVEs` workflow in the
nodejs-private/security-release repository, which runs when the
security release pull request is merged and holds until every patched
version is on nodejs.org/dist.

Update the security release runbook accordingly:

  * Note the OpenJS CNA option under 'Requesting CVEs', including the
    resume behaviour on supervised buckets.
  * Mark 'Request publication of H1 CVE requests' in cleanup as a no-op
    under the OpenJS CNA path.
  * Add a Post-Release step describing the automatic publication, the
    nodejs.org gate, the failure issue and the delay before cve.org.
  * Scope the HackerOne form under 'Updating CVEs' to HackerOne-issued
    ids and point OpenJS-issued ids at the CNA coordinators.

Refs: openjs-foundation/security-wg#337
Refs: nodejs/node-core-utils#1232
Refs: nodejs-private/security-release#87
Assisted-by: Claude
Signed-off-by: Ulises Gascon <ulisesgascongonzalez@gmail.com>
@UlisesGascon
UlisesGascon deleted the feat/openjs-cna-reserve branch November 6, 2026 11:45
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