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: missing certificate location on netbsd #9285

Closed
0-wiz-0 opened this issue Dec 12, 2014 · 5 comments
Closed

crypto/x509: missing certificate location on netbsd #9285

0-wiz-0 opened this issue Dec 12, 2014 · 5 comments
Assignees
Milestone

Comments

@0-wiz-0
Copy link

@0-wiz-0 0-wiz-0 commented Dec 12, 2014

Please add the default location for the certificate file on NetBSD:

diff --git a/src/crypto/x509/root_unix.go b/src/crypto/x509/root_unix.go
index f77d6c0..3b24347 100644
--- a/src/crypto/x509/root_unix.go
+++ b/src/crypto/x509/root_unix.go
@@ -13,6 +13,7 @@ var certFiles = []string{
        "/etc/ssl/certs/ca-certificates.crt",     // Debian/Ubuntu/Gentoo etc.
        "/etc/pki/tls/certs/ca-bundle.crt",       // Fedora/RHEL
        "/etc/ssl/ca-bundle.pem",                 // OpenSUSE
+       "/etc/openssl/certs/ca-certificates.crt", // NetBSD
        "/etc/ssl/cert.pem",                      // OpenBSD
        "/usr/local/share/certs/ca-root-nss.crt", // FreeBSD/DragonFly
        "/etc/pki/tls/cacert.pem",                // OpenELEC
@ianlancetaylor
Copy link
Contributor

@ianlancetaylor ianlancetaylor commented Dec 12, 2014

Can't even the *BSD systems agree on one location for these?

@davecheney
Copy link
Contributor

@davecheney davecheney commented Dec 13, 2014

@iant I'm increasingly frustrated with this situation. Would you be opposed
to a PR that split this definition into individual files ? At least that
way we can avoid stating a bunch of locations which will never be true?
On 13 Dec 2014 01:04, "Ian Lance Taylor" notifications@github.com wrote:

Can't even the *BSD systems agree on one location for these?


Reply to this email directly or view it on GitHub
#9285 (comment).

@bradfitz
Copy link
Contributor

@bradfitz bradfitz commented Dec 13, 2014

Sure. Feel free to +build it with an init function which calls registerCertPath(string) or something.

@davecheney
Copy link
Contributor

@davecheney davecheney commented Dec 13, 2014

Will do, I'll put it on my plate for when the tree reopens.
On 13 Dec 2014 13:31, "Brad Fitzpatrick" notifications@github.com wrote:

Sure. Feel free to +build it with an init function which calls
registerCertPath(string) or something.


Reply to this email directly or view it on GitHub
#9285 (comment).

@bradfitz bradfitz modified the milestone: Go1.5 Dec 16, 2014
@bradfitz bradfitz removed the release-go1.5 label Dec 16, 2014
@mikioh mikioh changed the title NetBSD: certificate location missing crypto/x509: missing certificate location on netbsd Dec 20, 2014
@davecheney davecheney self-assigned this Jan 5, 2015
@davecheney davecheney closed this in 6829229 Jan 5, 2015
@golang golang locked and limited conversation to collaborators Jun 25, 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
5 participants
You can’t perform that action at this time.