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

Update Mozilla CA certificate store to latest (10-16-2019) for libgit2 SSL. #33617

Merged
merged 3 commits into from
Oct 21, 2019

Conversation

mikhail-j
Copy link
Contributor

@staticfloat

The curl project has updated the Mozilla CA certificate store earlier last week (10/16/2019).

This update coincides with CA certificate list changes to Mozilla NSS 3.46.

2 CA certificates were removed:

  • Certplus Class 2 Primary CA
  • Deutsche Telekom Root CA 2

Although my pull request (#33106) from August was approved, the commits within that pull request were not merged upstream.

So I decided to merge the commits from the previous pull request into this pull request to prevent merge conflicts in the event both pull requests are merged upstream.

@StefanKarpinski StefanKarpinski added security System security concerns and vulnerabilities backport 1.0 priority This should be addressed urgently labels Oct 21, 2019
@staticfloat
Copy link
Member

Thanks, as always Mikhail!

@staticfloat staticfloat merged commit eb9d68a into JuliaLang:master Oct 21, 2019
@StefanKarpinski
Copy link
Member

In the future, let's always merge these any time someone seems one that is approved and passes CI. Also, always label for backport so that it doesn't get forgotten.

@mikhail-j mikhail-j deleted the libgit2-cacert-october-2019 branch October 22, 2019 01:35
@KristofferC KristofferC mentioned this pull request Oct 22, 2019
19 tasks
This was referenced Nov 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority This should be addressed urgently security System security concerns and vulnerabilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants