Sitelet https://web.archive.org/web/20220423220012/https://github.com/nodejs/node/pull/37070
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

tls: add ability to get cert/peer cert as X509Certificate object #37070

Closed
wants to merge 1 commit into from
Closed

tls: add ability to get cert/peer cert as X509Certificate object #37070

wants to merge 1 commit into from

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented Jan 26, 2021

Adds new getX509Certificate() and getPeerX509Certificate() as options on TLSSocket to get the cert/peercert as the new X509Certificate object.

Signed-off-by: James M Snell jasnell@gmail.com

@jasnell jasnell added tls crypto semver-minor labels Jan 26, 2021
@nodejs-github-bot nodejs-github-bot added c++ lib / src labels Jan 26, 2021
@nodejs-github-bot

This comment has been minimized.

Sign in to view
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Jan 26, 2021 •
edited by jasnell

CI: https://ci.nodejs.org/job/node-test-pull-request/35757/ 💚

tniessen
tniessen reviewed Jan 27, 2021
View changes
doc/api/crypto.md Outdated Show resolved Hide resolved
@jasnell jasnell requested a review from tniessen Jan 27, 2021
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Jan 27, 2021 •
edited by jasnell

CI: https://ci.nodejs.org/job/node-test-pull-request/35782/ 💚

@jasnell jasnell added the author ready label Jan 27, 2021
Trott
Trott reviewed Jan 31, 2021
View changes
doc/api/crypto.md Outdated Show resolved Hide resolved
Trott
Trott reviewed Jan 31, 2021
View changes
doc/api/tls.md Outdated Show resolved Hide resolved
@jasnell
tls: add ability to get cert/peer cert as X509Certificate object
89bae06 
Signed-off-by: James M Snell <jasnell@gmail.com>
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Jan 31, 2021

CI: https://ci.nodejs.org/job/node-test-pull-request/35841/

@tniessen
Copy link
Member

@tniessen tniessen commented Jan 31, 2021

CI is green. It would be great to get feedback from people with more experience with TLS than me, @nodejs/crypto. Apart from that, this is ready to land.

panva
panva approved these changes Feb 2, 2021
View changes
@jasnell
Copy link
Member Author

@jasnell jasnell commented Feb 2, 2021

Landed in c973d50

@jasnell jasnell closed this Feb 2, 2021
jasnell added a commit that referenced this issue Feb 2, 2021
@jasnell
tls: add ability to get cert/peer cert as X509Certificate object
c973d50 
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #37070
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
danielleadams pushed a commit that referenced this issue Feb 16, 2021
@jasnell @danielleadams
tls: add ability to get cert/peer cert as X509Certificate object
dc84c18 
Signed-off-by: James M Snell <jasnell@gmail.com>

PR-URL: #37070
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
danielleadams added a commit that referenced this issue Feb 16, 2021
@danielleadams
2021-02-16, Version 15.9.0 (Current)
5133659 
Notable Changes:

* crypto:
  * add keyObject.export() 'jwk' format option (Filip Skokan) #37081
* deps:
  * upgrade to libuv 1.41.0 (Colin Ihrig) #37360
* doc:
  * add dmabupt to collaborators (Xu Meng) #37377
  * refactor fs docs structure (James M Snell) #37170
* fs:
  * add fsPromises.watch() (James M Snell) #37179
  * use a default callback for fs.close() (James M Snell) #37174
  * add AbortSignal support to watch (Benjamin Gruenbaum) #37190
* perf_hooks:
  * introduce createHistogram (James M Snell) #37155
* stream:
  * improve Readable.from error handling (Benjamin Gruenbaum) #37158
* timers:
  * introduce setInterval async iterator (linkgoron) #37153
* tls:
  * add ability to get cert/peer cert as X509Certificate object (James M Snell) #37070
@danielleadams danielleadams mentioned this pull request Feb 16, 2021
Closed
@danielleadams danielleadams mentioned this pull request Feb 16, 2021
Merged
danielleadams added a commit that referenced this issue Feb 17, 2021
@danielleadams
2021-02-17, Version 15.9.0 (Current)
889ffde 
Notable Changes:

* crypto:
  * add keyObject.export() 'jwk' format option (Filip Skokan) #37081
* deps:
  * upgrade to libuv 1.41.0 (Colin Ihrig) #37360
* doc:
  * add dmabupt to collaborators (Xu Meng) #37377
  * refactor fs docs structure (James M Snell) #37170
* fs:
  * add fsPromises.watch() (James M Snell) #37179
  * use a default callback for fs.close() (James M Snell) #37174
  * add AbortSignal support to watch (Benjamin Gruenbaum) #37190
* perf_hooks:
  * introduce createHistogram (James M Snell) #37155
* stream:
  * improve Readable.from error handling (Benjamin Gruenbaum) #37158
* timers:
  * introduce setInterval async iterator (linkgoron) #37153
* tls:
  * add ability to get cert/peer cert as X509Certificate object (James M Snell) #37070
danielleadams added a commit that referenced this issue Feb 17, 2021
@danielleadams
2021-02-17, Version 15.9.0 (Current)
b471267 
Notable Changes:

* crypto:
  * add keyObject.export() jwk format option (Filip Skokan) #37081
* deps:
  * upgrade to libuv 1.41.0 (Colin Ihrig) #37360
* doc:
  * add dmabupt to collaborators (Xu Meng) #37377
  * refactor fs docs structure (James M Snell) #37170
* fs:
  * add fsPromises.watch() (James M Snell) #37179
  * use a default callback for fs.close() (James M Snell) #37174
  * add AbortSignal support to watch (Benjamin Gruenbaum) #37190
* perf_hooks:
  * introduce createHistogram (James M Snell) #37155
* stream:
  * improve Readable.from error handling (Benjamin Gruenbaum) #37158
* timers:
  * introduce setInterval async iterator (linkgoron) #37153
* tls:
  * add ability to get cert/peer cert as X509Certificate object (James M Snell) #37070
danielleadams added a commit that referenced this issue Feb 17, 2021
@danielleadams
2021-02-17, Version 15.9.0 (Current)
877f66e 
PR-URL: #37406

Notable Changes:

* crypto:
  * add keyObject.export() jwk format option (Filip Skokan) #37081
* deps:
  * upgrade to libuv 1.41.0 (Colin Ihrig) #37360
* doc:
  * add dmabupt to collaborators (Xu Meng) #37377
  * refactor fs docs structure (James M Snell) #37170
* fs:
  * add fsPromises.watch() (James M Snell) #37179
  * use a default callback for fs.close() (James M Snell) #37174
  * add AbortSignal support to watch (Benjamin Gruenbaum) #37190
* perf_hooks:
  * introduce createHistogram (James M Snell) #37155
* stream:
  * improve Readable.from error handling (Benjamin Gruenbaum) #37158
* timers:
  * introduce setInterval async iterator (linkgoron) #37153
* tls:
  * add ability to get cert/peer cert as X509Certificate object (James M Snell) #37070
danielleadams added a commit that referenced this issue Feb 18, 2021
@danielleadams
2021-02-17, Version 15.9.0 (Current)
3bf50a7 
PR-URL: #37406

Notable Changes:

* crypto:
  * add keyObject.export() jwk format option (Filip Skokan) #37081
* deps:
  * upgrade to libuv 1.41.0 (Colin Ihrig) #37360
* doc:
  * add dmabupt to collaborators (Xu Meng) #37377
  * refactor fs docs structure (James M Snell) #37170
* fs:
  * add fsPromises.watch() (James M Snell) #37179
  * use a default callback for fs.close() (James M Snell) #37174
  * add AbortSignal support to watch (Benjamin Gruenbaum) #37190
* perf_hooks:
  * introduce createHistogram (James M Snell) #37155
* stream:
  * improve Readable.from error handling (Benjamin Gruenbaum) #37158
* timers:
  * introduce setInterval async iterator (linkgoron) #37153
* tls:
  * add ability to get cert/peer cert as X509Certificate object (James M Snell) #37070
@targos targos added dont-land-on-v12.x dont-land-on-v14.x labels Aug 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Trott Trott

@panva panva

@tniessen tniessen

Assignees
No one assigned
Labels
author ready c++ crypto dont-land-on-v12.x dont-land-on-v14.x lib / src semver-minor tls
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@jasnell @nodejs-github-bot @tniessen @panva @Trott @targos