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

mongo database engine does not support tls_ca #980

Open
hotpheex opened this issue Feb 15, 2021 · 2 comments
Open

mongo database engine does not support tls_ca #980

hotpheex opened this issue Feb 15, 2021 · 2 comments

Comments

@hotpheex
Copy link

Hi there,

Thank you for opening an issue. Please note that we try to keep the Terraform issue tracker reserved for bug reports and feature requests. For general usage questions, please see: https://www.terraform.io/community.html.

Terraform Version

0.13.5

Affected Resource(s)

Please list the resources as a list, for example:

  • vault_database_secret_backend_connection

Terraform Configuration Files

resource "vault_database_secret_backend_connection" "database" {
  backend       = var.database_path
  name          = var.database_name
  allowed_roles = ["*"]

  mongodb {
    connection_url = var.database_connection_url
    tls_ca = var.database_tls_ca
  }
}

Debug Output

Error: Unsupported argument

  on modules/.../document_db_config.tf line 14, in resource "vault_database_secret_backend_connection" "database":
  14:     tls_ca = var.database_tls_ca

An argument named "tls_ca" is not expected here.

Expected Behavior

Configures the tls_ca setting in the mongodb plugin https://www.vaultproject.io/api-docs/secret/databases/mongodb#parameters

Actual Behavior

The provider does not recognise the tls_ca key as a valid option

Steps to Reproduce

terraform validate

References

@afourneau
Copy link

Any update on this?

@vbyrd
Copy link

vbyrd commented Mar 22, 2023

Any update on this? Documentdb by default enables tls so it is beneficial to have these options added. It looks like #882 has been sitting for a while.

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

Successfully merging a pull request may close this issue.

3 participants