From 89081b087bf23f9102e6642f6f666d423b59d4a6 Mon Sep 17 00:00:00 2001 From: Vishal Date: Mon, 14 Jan 2019 16:53:06 +0530 Subject: [PATCH] Changed Kafka topic name validation (#1051) * Changed Kafka topic name validation so that capital characters are also allowed * Removing support for \ in Kafka topic name * Using raw string some specific sequences --- pkg/apis/fission.io/v1/validation.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/pkg/apis/fission.io/v1/validation.go b/pkg/apis/fission.io/v1/validation.go index ff783d06..c1db1c6f 100644 --- a/pkg/apis/fission.io/v1/validation.go +++ b/pkg/apis/fission.io/v1/validation.go @@ -36,7 +36,8 @@ const ( var ( validAzureQueueName = regexp.MustCompile("^[a-z0-9][a-z0-9\\-]*[a-z0-9]$") - validKafkaTopicName = regexp.MustCompile("^[a-z0-9][a-z0-9\\-._]*[a-z0-9]$") + // Need to use raw string to support escape sequence for - & . chars + validKafkaTopicName = regexp.MustCompile(`^[a-zA-Z0-9][a-zA-Z0-9\-\._]*[a-zA-Z0-9]$`) ) type (