Merge pull request #5051 from dokku/5044-sluggify-vector

Ensure vector component sources have valid names

plugins/logs/functions.go

@@ -167,6 +167,7 @@ func writeVectorConfig() error {
 			continue
 		}
 
+		appName = strings.ReplaceAll(appName, ".", "-")
 		sink, err := sinkValueToConfig(appName, value)
 		if err != nil {
 			return err

tests/unit/logs.bats

@@ -522,6 +522,11 @@ teardown() {
   assert_failure
   assert_output_contains "Vector container does not exist"
 
+  run /bin/bash -c "dokku apps:create example.com"
+  echo "output: $output"
+  echo "status: $status"
+  assert_success
+
   run /bin/bash -c "dokku logs:vector-start 2>&1"
   echo "output: $output"
   echo "status: $status"
@@ -534,6 +539,11 @@ teardown() {
   assert_success
   assert_output_contains "Vector container logs"
 
+  run /bin/bash -c "dokku --force apps:destroy example.com"
+  echo "output: $output"
+  echo "status: $status"
+  assert_success
+
   run /bin/bash -c "dokku logs:vector-logs --num 10 2>&1"
   echo "output: $output"
   echo "status: $status"