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: allow cert bundle path to be set by environment variable #3905

Closed
gopherbot opened this issue Aug 4, 2012 · 7 comments
Closed

crypto/x509: allow cert bundle path to be set by environment variable #3905

gopherbot opened this issue Aug 4, 2012 · 7 comments
Milestone

Comments

@gopherbot
Copy link

@gopherbot gopherbot commented Aug 4, 2012

by viriketo:

Using go 1.0.2 on linux-x86_64.

It looks like go https works with a hardcoded path to ca-bundle list of accepted
certificates.

I think it would be nice if src/pkg/crypto/x509/root_unix.go could look at some
environment variable for an exclusive ca-cert.

In the GNU/Linux distribution NixOS we have the system ca-cert bundle sitting at a path
different than those hardcoded in root_unix.go, and I can patch go1.0.2 to make it work.
But users may want to have control about the trusted certificates, when they run go
programs.
@robpike
Copy link
Contributor

@robpike robpike commented Aug 4, 2012

Comment 1:

Labels changed: added priority-later, packagechange, removed priority-triage.

Owner changed to @agl.

Status changed to Accepted.

@rsc
Copy link
Contributor

@rsc rsc commented Nov 27, 2013

Comment 3:

Labels changed: added go1.3maybe.

@rsc
Copy link
Contributor

@rsc rsc commented Dec 4, 2013

Comment 4:

Labels changed: added release-none, removed go1.3maybe.

@rsc
Copy link
Contributor

@rsc rsc commented Dec 4, 2013

Comment 5:

Labels changed: added repo-main.

@gopherbot
Copy link
Author

@gopherbot gopherbot commented Nov 21, 2014

Comment 6 by sokolyuk:

Please add
"/etc/ssl/cacert.pem",                      // OmniOS
into  pkg/crypto/x509/root_unix.go
@odeke-em
Copy link
Member

@odeke-em odeke-em commented Oct 10, 2016

Any thoughts on this @agl?

@gopherbot
Copy link
Author

@gopherbot gopherbot commented Feb 2, 2017

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

@gopherbot gopherbot closed this in e83bcd9 May 3, 2017
@golang golang locked and limited conversation to collaborators May 3, 2018
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
4 participants
You can’t perform that action at this time.