#1894 Only limit scan name if it is too long

The function to generate scan names trims the name if it is too long, during tests I had it happen that it was short enough and then the program crashes with a panic because the index is out of range. This checks the length before to prevent the panic. Signed-off-by: Lukas Fischer <lukas.fischer@iteratec.com>
secureCodeBox · Sep 22, 2023 · a45cce0 · a45cce0
1 parent d0b38b7
commit a45cce0
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/auto-discovery/cloud-aws/kubernetes/kubernetes.go b/auto-discovery/cloud-aws/kubernetes/kubernetes.go
@@ -314,8 +314,12 @@ func getScanName(req Request, name string) string {
 	result = strings.ReplaceAll(result, ".", "-")
 	result = strings.ReplaceAll(result, "/", "-")
 
-	//limit scan name length to kubernetes limits
-	return result[:62]
+	// limit scan name length to kubernetes limits
+	if len(result) > 62 {
+		result = result[:62]
+	}
+
+	return result
 }
 
 func GetClient() (client.Client, string, error) {