Skip to content
This repository has been archived by the owner on Feb 5, 2020. It is now read-only.

Terraform should bcrypt adminPassword #643

Closed
ggreer opened this issue May 10, 2017 · 12 comments
Closed

Terraform should bcrypt adminPassword #643

ggreer opened this issue May 10, 2017 · 12 comments

Comments

@ggreer
Copy link
Contributor

ggreer commented May 10, 2017

Currently, the user has to provide the bcrypted, base64-encoded password. We also have to calculate this in the GUI. Doing this in terraform would make for a better experience for users.

@ggreer ggreer added this to the Theme: Overall cleanup and stability milestone May 10, 2017
@sym3tri
Copy link
Contributor

sym3tri commented May 11, 2017

Unfortunately that feature doesn't exist in Terraform Upstream. We could add it but it might take some time.

@ggreer
Copy link
Contributor Author

ggreer commented May 11, 2017

OK. Until then, we can do it in JS.

@sym3tri
Copy link
Contributor

sym3tri commented May 12, 2017

@ggreer we have someone who will be adding this upstream soon.

@sym3tri sym3tri modified the milestones: Sprint 3: Continued Test Automation, Sprint 2: Overall cleanup and stability May 12, 2017
@sym3tri
Copy link
Contributor

sym3tri commented May 26, 2017

Upstream PR hashicorp/terraform#14725

@sym3tri sym3tri modified the milestones: Sprint 4, Sprint 3: Continued Test Automation May 26, 2017
@radeksimko
Copy link

Just FYI - the linked PR was just merged and will be part of 0.9.7 release.

@sym3tri
Copy link
Contributor

sym3tri commented May 30, 2017

@radeksimko Nice! Thanks for the update.

@Quentin-M
Copy link
Contributor

Quentin-M commented Jun 19, 2017

Upstream was validated, the backend change can happen very quickly, as long as we get to update Terraform.

@Quentin-M
Copy link
Contributor

It will take another 3 weeks for Terraform 0.9.10 to be released, which is likely be the next version we use. Pushing for now.

@Quentin-M Quentin-M modified the milestones: Sprint 5, Sprint 4 Jun 21, 2017
@radeksimko
Copy link

@Quentin-M there are no more 0.9 releases planned at this point. The next planned release is 0.10.0beta1, but the bcrypt function was already released in 0.9.7 as mentioned in the changelog: https://github.com/hashicorp/terraform/blob/master/CHANGELOG.md#097-june-7-2017

@Quentin-M
Copy link
Contributor

I guess that is what I meant, 0.10, not 0.9 :) Thanks for reaching out!

For what it's worth, we do not plan to update to 0.9.7 nor 0.9.8 as they both suffer from a regression in template_file, which we use quite a bit.

@radeksimko
Copy link

Understood, sorry for the regression. 🙈

Just in case you missed this - there's a blogpost about planned changes which may affect you in 0.10:
https://www.hashicorp.com/blog/upcoming-provider-changes-in-terraform-0-10/

@Quentin-M
Copy link
Contributor

Quentin-M commented Jun 22, 2017

Thanks a lot, truly appreciate it! We are all looking forward to it!

@sym3tri sym3tri modified the milestones: Sprint 5, Sprint 6 Jun 30, 2017
@sym3tri sym3tri modified the milestone: Sprint 6 Aug 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants