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

crypto/x509: root_unix.go: crypto/x509: failed to load system roots and no roots provided. #9146

Closed
gopherbot opened this issue Nov 21, 2014 · 4 comments
Milestone

Comments

@gopherbot
Copy link

@gopherbot gopherbot commented Nov 21, 2014

by sokolyuk:

Please add

        "/etc/ssl/cacert.pem",                    // OmniOS

into pkg/crypto/x509/root_unix.go
@ianlancetaylor
Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Nov 21, 2014

Comment 1:

Labels changed: added repo-main, release-go1.5.

@minux
Copy link
Member

@minux minux commented Nov 21, 2014

Comment 2:

could all these illumos-based opensolaris derivatives standardize on a single root
certificates
location?
smartos uses /etc/ssl/certs/ca-certificates.crt, which is also used by major Linux
distributions,
and Solaris 11.2+ standardized on /etc/certs/ca-certificates.crt.
@davecheney
Copy link
Contributor

@davecheney davecheney commented Nov 21, 2014

Comment 3:

As someone who's campaigned for a few additions to this file over time I admit that my
tolerance for the unlimited number of locations that distros hide this file is exhausted.
I think if this bug is fixed in 1.5 some time should be spent to split out the slice of
locations to search to be platform specific. At least this puts the mess in smaller
boxes.
@gopherbot gopherbot added new labels Nov 21, 2014
@bradfitz bradfitz modified the milestone: Go1.5 Dec 16, 2014
@rsc rsc removed the repo-main label Apr 14, 2015
@gopherbot
Copy link
Author

@gopherbot gopherbot commented Jul 15, 2015

CL https://golang.org/cl/12203 mentions this issue.

@rsc rsc closed this in f8c97ab Jul 15, 2015
@golang golang locked and limited conversation to collaborators Jul 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
6 participants
You can’t perform that action at this time.