Skip to content

Conversation

@DMeechan
Copy link
Contributor

@DMeechanDMeechan commented Nov 7, 2023

Updated Crypto API definition for x509.keyUsage to x509.extKeyUsage.

Refs: #48727

Created at NodeConf! ❤️

@nodejs-github-botnodejs-github-bot added crypto Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations. labels Nov 7, 2023
@marco-ippolitomarco-ippolito added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Nov 7, 2023
@QardQard added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 7, 2023
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 7, 2023
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Nov 9, 2023

@jasnelljasnell added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 11, 2023
@nodejs-github-botnodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Nov 11, 2023
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/50603 ✔ Done loading data for nodejs/node/pull/50603 ----------------------------------- PR info ------------------------------------ Title doc: update Crypto API doc for x509.keyUsage to x509.extKeyUsage (#50603) Author Daniel Meechan (@DMeechan, first-time contributor) Branch DMeechan:crypto-api-key-usage -> nodejs:main Labels crypto, doc, code-and-learn Commits 1 - doc: update Crypto API doc for x509.keyUsage Committers 1 - Daniel Meechan PR-URL: https://github.com/nodejs/node/pull/50603 Refs: https://github.com/nodejs/node/issues/48727 Reviewed-By: James M Snell ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/50603 Refs: https://github.com/nodejs/node/issues/48727 Reviewed-By: James M Snell -------------------------------------------------------------------------------- ℹ This PR was created on Tue, 07 Nov 2023 15:15:49 GMT ✔ Approvals: 1 ✔ - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/50603#pullrequestreview-1726076103 ℹ This PR is being fast-tracked because it is from a Code and Learn event ✘ This PR needs to wait 71 more hours to land (or 0 hours if there is one more approval) ✔ Last GitHub CI successful ℹ Green GitHub CI is sufficient -------------------------------------------------------------------------------- ✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/6835039755

@marco-ippolitomarco-ippolito added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Nov 20, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 20, 2023
@nodejs-github-botnodejs-github-bot merged commit a9a4778 into nodejs:mainNov 20, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in a9a4778

targos pushed a commit that referenced this pull request Nov 23, 2023
PR-URL: #50603 Refs: #48727 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
martenrichter pushed a commit to martenrichter/node that referenced this pull request Nov 26, 2023
PR-URL: nodejs#50603 Refs: nodejs#48727 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
lucshi pushed a commit to lucshi/node that referenced this pull request Nov 27, 2023
PR-URL: nodejs#50603 Refs: nodejs#48727 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
@RafaelGSSRafaelGSS mentioned this pull request Nov 28, 2023
RafaelGSS pushed a commit that referenced this pull request Nov 29, 2023
PR-URL: #50603 Refs: #48727 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
@DMeechanDMeechan deleted the crypto-api-key-usage branch November 30, 2023 13:03
RafaelGSS pushed a commit that referenced this pull request Nov 30, 2023
PR-URL: #50603 Refs: #48727 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
UlisesGascon pushed a commit that referenced this pull request Dec 11, 2023
PR-URL: #50603 Refs: #48727 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
@UlisesGasconUlisesGascon mentioned this pull request Dec 12, 2023
UlisesGascon pushed a commit that referenced this pull request Dec 19, 2023
PR-URL: #50603 Refs: #48727 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
@panva
Copy link
Member

panva commented Aug 2, 2025

Why did this land? There's no extKeyUsage as far as I can tell...

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learnIssues related to the Code-and-Learn events and PRs submitted during the events.cryptoIssues and PRs related to the crypto subsystem.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@DMeechan@nodejs-github-bot@panva@jasnell@marco-ippolito@Qard