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

Windows jumpbox VM passwords are occasionally invalid #247

Closed
glennmusa opened this issue Jun 9, 2021 · 1 comment · Fixed by #284
Closed

Windows jumpbox VM passwords are occasionally invalid #247

glennmusa opened this issue Jun 9, 2021 · 1 comment · Fixed by #284
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@glennmusa
Copy link
Contributor

Description

We use Terraform's password provider to generate credentials for the jumpbox VMs. Occasionally, that random value will not satisfy password requirements:

Original Error: Code="InvalidParameter" 

Message="The supplied password must be between 8-123 characters long and must satisfy at least 3 of password complexity requirements from the following:

1) Contains an uppercase character
2) Contains a lowercase character
3) Contains a numeric digit
4) Contains a special character
5) Control characters are not allowed

Steps to Reproduce
Steps to reproduce the behavior:

  1. Deploy MLZ with bastion host jumpboxes (the default behavior)
  2. Get unlucky

Expected behavior
Passwords that are generated meet Windows credential requirements

Actual behavior
Occasionally you'll get a password that does not meet those requirements

Screenshots

Additional context
Operating System: n/a
Terraform Version: 0.13.4
Cloud (public, Azure Government, etc.): n/a

@glennmusa glennmusa added the bug Something isn't working label Jun 9, 2021
@glennmusa glennmusa self-assigned this Jun 9, 2021
@glennmusa glennmusa removed their assignment Jul 6, 2021
@glennmusa
Copy link
Contributor Author

Lisa pointed us to the optional schema for the resource to ensure the right bits are generated https://registry.terraform.io/providers/hashicorp/random/latest/docs/resources/password#optional

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant