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

Understanding the deprecation of client_auth argument #1843

Open
sebastienbonami opened this issue May 4, 2023 · 2 comments
Open

Understanding the deprecation of client_auth argument #1843

sebastienbonami opened this issue May 4, 2023 · 2 comments
Labels

Comments

@sebastienbonami
Copy link

Hi, in the docs of the latest version (3.15.2 as of today), we can read the following statement about the client_auth argument:

At present there is little reason to set this, because Terraform does not support the TLS certificate authentication mechanism. *Deprecated, use auth_login_cert instead.

The thing is, I don't see auth_login_cert being an alternative to client_auth so I'm trying to figure this out. auth_login_cert is for presenting a certificate for the cert authentication engine. client_auth is for presenting a certificate for the tcp listener as far as I understand.

Actually, client_auth is certainly needed if the tls_require_and_verify_client_cert parameter of the tcp listener is enabled, and it's not deprecated as far as I know.

So why deprecating the client_auth argument?

Thanks for the help.

@fairclothjm
Copy link
Contributor

@sebastienbonami Thanks for reporting! I don't have the context as to why the deprecation was announced. We are going to reevaluate this deprecation. Related to #2130

@avanschie
Copy link

Hi, I just stumbled upon this issue. I'm using a proxy to provide zero trust access to vault - there I need to present the TLS certificates to the proxy, i.e., I rely on client_auth. auth_login_cert doesn't look like an alternative and the removal of client_auth would break my setup!

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

No branches or pull requests

4 participants