Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Update root certificates #25829

Closed
wants to merge 2 commits into from
Closed

Update root certificates #25829

wants to merge 2 commits into from

Conversation

vbfox
Copy link

@vbfox vbfox commented Aug 9, 2015

The current root certificate bundle is out-of-date. I noticed it while I wanted to use https://keybase.io tools to certifiy my domain name.

This pull request update the $url that is used only as documentation in mk-ca-bundle.pl with the correct one (The URL changed when mozilla moved to Mercurial)

And I also updated the certificate bundle itself with the version aa275ad846f1 (Same as Firefox 39, latest change on 2015-04-20)

There is no code to download it automatically but the mk-ca-bundle.pl
file document the url at the top.

The url documented is the one before mozilla moved to mercurial as VCS
and wasn't updated for 3 years. The new one is the certificate bundle
for the current stable release of Firefox.
Update the certificate data in certdata.txt and node_root_certs.h with
the current Mozilla version.

Version is aa275ad846f1 with latest update on 2015-04-20 as distributed
in Firefox 39.
@vbfox
Copy link
Author

vbfox commented Aug 11, 2015

Seem like the corresponding update was already in io.js : nodejs/node#1833

Depending on the timeline for the unified version it would be interesting to update the certificate in nodej before the merge to avoid giving errors on valid https websites.

@jasnell
Copy link
Member

jasnell commented Aug 11, 2015

@jasnell
Copy link
Member

jasnell commented Aug 16, 2015

@vbfox this would need to be opened as a PR against the v0.12 branch. Otherwise, we'll pick up the changes from nodejs/node in the converged repo. I recommend closing this particular PR and if you'd like to pursue a change to v0.12, open a new one targeted there please :-) Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants