Skip to content

Don't show symbols while input if variable marked as sensitive#29520

Merged
jbardin merged 2 commits intohashicorp:mainfrom
ComBin:main
Dec 16, 2022
Merged

Don't show symbols while input if variable marked as sensitive#29520
jbardin merged 2 commits intohashicorp:mainfrom
ComBin:main

Conversation

@ComBin
Copy link
Copy Markdown
Contributor

@ComBin ComBin commented Sep 3, 2021

For example openstack admin password

@hashicorp-cla
Copy link
Copy Markdown

hashicorp-cla commented Sep 3, 2021

CLA assistant check
All committers have signed the CLA.

@crw
Copy link
Copy Markdown
Contributor

crw commented May 20, 2022

Thanks for this submission! We usually require a contribution to have a use case and solution discussed with the maintainers in a Github issue before we will review a pull request. It looks like this PR is similar to this issue: #29609 -- is that correct? I also noticed you seem to have added a binary file in your commit which I believe is a terraform binary - that would need to be removed.

Please let me know if this PR is still relevant and what the use case is that it is solving. Thanks again for the submission!

@crw crw added enhancement waiting-response An issue/pull request is waiting for a response from the community labels May 20, 2022
@vercel
Copy link
Copy Markdown

vercel bot commented May 21, 2022

@ComBin is attempting to deploy a commit to the HashiCorp Team on Vercel.

A member of the Team first needs to authorize it.

@ComBin
Copy link
Copy Markdown
Contributor Author

ComBin commented May 21, 2022

Hello, @crw! Yes, #29609 looks similar. Also i removed binary file from PR.
And case for example, entering a password value for openstack administrator account. While value password marked as sensitive before patch, entering looks like:

$ terraform plan
Acquiring state lock. This may take a few moments...
var.os_passwd
  Openstack admin password

  Enter a value: sdfsfsfsf

After:

$ terraform plan
var.os_passwd
  Openstack admin password

  Enter a value: 

Behavior as while typing SSH password.

@crw crw removed the waiting-response An issue/pull request is waiting for a response from the community label May 23, 2022
@crw
Copy link
Copy Markdown
Contributor

crw commented May 23, 2022

Thanks! I appreciate that response. I'll put it on the triage queue!

@jbardin jbardin self-assigned this Dec 13, 2022
@jbardin jbardin merged commit 3cda7a0 into hashicorp:main Dec 16, 2022
@github-actions
Copy link
Copy Markdown
Contributor

Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch.

@github-actions
Copy link
Copy Markdown
Contributor

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants