Skip to content
This repository has been archived by the owner on Jan 4, 2021. It is now read-only.

Shouldn't require certs to be stored in filesystem #3

Open
snej opened this issue May 30, 2013 · 1 comment
Open

Shouldn't require certs to be stored in filesystem #3

snej opened this issue May 30, 2013 · 1 comment

Comments

@snej
Copy link

snej commented May 30, 2013

The New() function takes filesystem paths to the certificate and private key files. That's too limiting — the server may not have filesystem storage available, or the private key may be considered too sensitive so it might be kept in a Keychain-like store or an LDAP server.

Probably the most flexible way to do this would be for New() to take a tls.Certificate as a parameter, and let the caller load the certificate by calling either LoadX509KeyPair or X509KeyPair, the latter of which takes the components as raw byte arrays.

@googollee
Copy link
Collaborator

Yep. I'll change it.

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

No branches or pull requests

2 participants