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

azurerm provider should define kube_config_raw as sensitive #1220

Closed
ghost opened this issue May 9, 2018 · 6 comments · Fixed by #1225
Closed

azurerm provider should define kube_config_raw as sensitive #1220

ghost opened this issue May 9, 2018 · 6 comments · Fixed by #1225

Comments

@ghost
Copy link

ghost commented May 9, 2018

This issue was originally opened by @subesokun as hashicorp/terraform#18013. It was migrated here as a result of the provider split. The original body of the issue is below.


Terraform Version

Terraform v0.11.7
provider.azurerm: version = "~> 1.4"

Debug Output

https://gist.github.com/subesokun/ae9893a093c4ce7fcdebf2cb5cc95c0d

Expected Behavior

If a kubernetes cluster gets re-deployed the kube_config_raw content shouldn't be visible in the console log and be marked as <sensitive>

Actual Behavior

kube_config_raw content gets printed in clear text into the console log.

Steps to Reproduce

  1. terraform init
  2. terraform apply
  3. Change some attribute like the linux_profile.ssh_key
  4. terraform apply

Additional Context

We're deploying our kubernetes clusters as part of our CI/CD pipelines and usually every developer in the project has access to the deployment logs of those pipelines and hence we need to keep the console logs clean from any sensitive data. Now as the kube_config_raw gets logged in clear text into the console this is very critical to us as everybody with access to the logs could access the cluster and compromise it which we have to prevent by all means.

@subesokun
Copy link

Awesome, thanks a lot!

@katbyte
Copy link
Collaborator

katbyte commented May 14, 2018

Hey @subesokun

Just a heads up that we have released v1.5.0 of the provider that includes this fix 🙂

@subesokun
Copy link

@katbyte thanks for the heads up! Unfortunately I've noticed today that also in the azurerm_storage_account the primary access key, secondary access key and the connection strings are getting printed in clear text in the execution plan if the storage account needs to be recreated. So if you don't execute that plan immediately you'll face a security risk. But I guess I should open another issue for it :)

@subesokun
Copy link

#1239

@katbyte
Copy link
Collaborator

katbyte commented May 15, 2018

Yep! Thanks for letting us know 🙂

@ghost
Copy link
Author

ghost commented Mar 31, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@ghost ghost locked and limited conversation to collaborators Mar 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants