documentation - backup_vault_aws - Fix length validation error handling for vault_name parameter #26127
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello!
Usecase:
in
backup_aws_vault
, if you set the variablevault_name
with a string of more than 50 characters, instead of a length error, it raises:"must consist of lowercase letters, numbers, and hyphens."
I think it should raise something along the line of:
expected length of name to be in the range (1 - 50)
I think this regex is to blame (the last part of the regex):
^[a-zA-Z0-9\-\_\.]{1, 50}$
Proposed solution:
Split the validation into syntax validation and length validation
PS: I might edit this PR later on to comply with guidelines (meanwhile I let this PR in draft).
Community Note
Relates OR Closes #0000
Output from acceptance testing: