Skip to content
This repository was archived by the owner on Jan 16, 2025. It is now read-only.

Commit a952b91

Browse files
authored
fix(syncer): S3 bucket logging prefix variable condition (#3251)
fix: S3 bucket logging prefix regex
1 parent e679021 commit a952b91

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

modules/runner-binaries-syncer/variables.tf

+3-3
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,10 @@ variable "s3_logging_bucket_prefix" {
4949
type = string
5050
default = null
5151

52-
# Make sure the bucket name only contains legal characters
52+
# Make sure the bucket prefix only contains legal characters
5353
validation {
54-
error_message = "Only lowercase alphanumeric characters and hyphens allowed in the bucket name."
55-
condition = var.s3_logging_bucket_prefix == null || can(regex("^[a-z0-9-]*$", var.s3_logging_bucket_prefix))
54+
error_message = "Only alphanumeric characters, hyphens followed by single slashes allowed in the bucket prefix."
55+
condition = var.s3_logging_bucket_prefix == null || can(regex("^(([a-zA-Z0-9-])+(\\/?))*$", var.s3_logging_bucket_prefix))
5656
}
5757
}
5858

0 commit comments

Comments
 (0)