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

random_string and random_password require checks on length #233

Closed
1 task done
bendbennett opened this issue Apr 21, 2022 · 1 comment · Fixed by #234
Closed
1 task done

random_string and random_password require checks on length #233

bendbennett opened this issue Apr 21, 2022 · 1 comment · Fixed by #234

Comments

@bendbennett
Copy link
Contributor

Terraform CLI and Provider Versions

Terraform v1.1.8
terraform-provider-random v3.1.2

Use Cases or Problem Statement

If length used with random_string or random_password is less than 1 this gives rise to the following problems:

length < 0

If length is < 0 then the provider panics:

length == 0

If length == 0 the provider generates a somewhat ambiguous error message:

Proposal

Add checks to ensure that length is > 0.

How much impact is this issue causing?

Low

Additional Information

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
Copy link

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 have found a problem that seems similar to this, 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 May 24, 2024
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.

1 participant