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

Please upgrade node_root_certs.h for 0.x versions #7265

Closed
m-bucher opened this issue Jun 10, 2016 · 6 comments
Closed

Please upgrade node_root_certs.h for 0.x versions #7265

m-bucher opened this issue Jun 10, 2016 · 6 comments
Labels
tls Issues and PRs related to the tls subsystem.

Comments

@m-bucher
Copy link

  • Version: 0.x
  • Platform: any (tested on CentOS 6)
  • Subsystem: core

I am using using Node.js 0.10 for a service that tries to connect to customer systems and returns errors if there are any problems (e.g. concerning the Certificates).

Problem is I get 'self-signed certificate in chain' error for a server cert signed by "COMODO RSA Certification Authority". because that CA-certificate is not present in Node.js 0.10 or even 0.12.

PS: Yes, I know I could update Node.js, but let's say I cannot for reasons I do not want to explain here ;-)

@silverwind silverwind added v0.10 lib / src Issues and PRs related to general changes in the lib or src directory. labels Jun 10, 2016
@silverwind
Copy link
Contributor

Also missing in v4.x.

@m-bucher
Copy link
Author

Best solution would be to use the OS'es certificate trust-store ;-)

@MylesBorins
Copy link
Contributor

/cc @nodejs/lts

@mscdex mscdex added tls Issues and PRs related to the tls subsystem. and removed lib / src Issues and PRs related to general changes in the lib or src directory. labels Jun 10, 2016
@bnoordhuis
Copy link
Member

#7363

@jasnell
Copy link
Member

jasnell commented Oct 26, 2016

This appears to have been done for all release lines from v4 forward. Given that it is unlikely that we will cut a new v0.10 release before EOL, I will pull the v0.10 label from this. We need to decide still if we're going to pull this into v0.12 at all.

@jasnell jasnell removed the v0.10 label Oct 26, 2016
@jasnell
Copy link
Member

jasnell commented Dec 29, 2016

Closing this given that support for v0.12 is coming to a close in two days and there will not be any further updates to the v0.12 branch.

@jasnell jasnell closed this as completed Dec 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tls Issues and PRs related to the tls subsystem.
Projects
None yet
Development

No branches or pull requests

6 participants