Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add validation for netmask length #1321

Merged
merged 1 commit into from
Jan 27, 2021
Merged

Add validation for netmask length #1321

merged 1 commit into from
Jan 27, 2021

Conversation

koikonom
Copy link
Contributor

@koikonom koikonom commented Jan 27, 2021

Ensures a network mask's length will never exceed 32 for IPv4 and 128
for IPv6.

Fixes #1316

@koikonom koikonom requested a review from aareet January 27, 2021 17:22
@ghost ghost added the size/xs Relative Sizing: Extra-Small label Jan 27, 2021
Ensures a network mask's length will never exceed 32 for IPv4 and 128
for IPv6.
@koikonom koikonom merged commit b2f6e3e into master Jan 27, 2021
@koikonom koikonom deleted the validate_netmask branch January 27, 2021 18:13
@ghost
Copy link

ghost commented Feb 27, 2021

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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@ghost ghost locked as resolved and limited conversation to collaborators Feb 27, 2021
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
size/xs Relative Sizing: Extra-Small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

terraform 0.14.5 crash with vSphere provider 1.24.3
2 participants