Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

parametize minimum_password_length in IAM setup module #258

Open
ketzacoatl opened this issue Oct 3, 2019 · 1 comment
Open

parametize minimum_password_length in IAM setup module #258

ketzacoatl opened this issue Oct 3, 2019 · 1 comment
Assignees
@psibi
Labels
help wanted P1 starter An issue that might be good for beginners to Terraform or foundation

Comments

@ketzacoatl
Copy link
Contributor

This here ought to be a variable with a default to 12: https://github.com/fpco/terraform-aws-foundation/blob/master/modules/setup-meta-infrastructure/password-policy.tf#L7

Consider parametizing other attributes where that makes sense, but this minimum password length is required.
@ketzacoatl ketzacoatl added help wanted P1 starter An issue that might be good for beginners to Terraform or foundation labels Oct 3, 2019
@psibi psibi self-assigned this Oct 3, 2019
@psibi
Copy link
Member

psibi commented Oct 3, 2019

PR raised here: #259

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@psibi psibi

Labels
help wanted P1 starter An issue that might be good for beginners to Terraform or foundation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@psibi @ketzacoatl