Support for self-signed certificates #1609
Labels
⌛ pending author's response
Requested additional information from the reporter
✨ enhancement
New feature or request
Is your feature request related to a problem? Please describe.
To my understanding the project doesn't support self-signed certificates.
The only support for TLS on the node side I know of is
insecure = false
, i.e. verifying the certificate against the normal root chains, orinsecure = true
, i.e. not verifying the certificate at all. Furthermore, I found no documentation on that matter and all examples just useinsecure = true
.Describe the solution you'd like
Ideally, when setting up the provider, on the same level as
insecure
another optional argument should exist which takes either a path to a certificate or the certificate itself. The corresponding certificate should be used to verify the node's identity.Describe alternatives you've considered
Only providing the SHA256sum of the certificate might be enough, however, I'm not sure if the TLS library can handle that input.
Additional context
The relevant place in the code:
terraform-provider-proxmox/proxmox/api/client.go
Lines 87 to 94 in 9d166b9
To my understanding, this only sets
true
orfalse
according to the value given here:terraform-provider-proxmox/proxmoxtf/provider/provider.go
Line 100 in 9d166b9
I think it's important to be able to use TLS, especially outside of purely testing environments.
As a default Proxmox install comes with a self-signed certificate, this should, in my opinion, be the first point to start.
Especially, as not everyone set's up a proper certificate chain with an approved root certificate.
The text was updated successfully, but these errors were encountered: