fix: make sure names are kubernetes compatible names

Fixes #723
kaoto-archive · Jun 28, 2023 · b6915b8 · b6915b8
1 parent 411a91e
commit b6915b8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/api/src/main/java/io/kaoto/backend/api/resource/v2/IntegrationsResource.java b/api/src/main/java/io/kaoto/backend/api/resource/v2/IntegrationsResource.java
@@ -146,7 +146,8 @@ private static void ensureUniqueNames(FlowsWrapper answer) {
             }
             //Make sure there is an id/name assigned to all flows
             if (!flow.getMetadata().containsKey(name)) {
-                flow.getMetadata().put(name, flow.getDsl() + System.currentTimeMillis());
+                flow.getMetadata().put(name,
+                        flow.getDsl().toLowerCase().replaceAll(" ", "") + System.currentTimeMillis());
             }
             //Make sure it is unique
             if (usedIds.contains(flow.getMetadata().get(name))) {

diff --git a/api/src/test/java/io/kaoto/backend/api/resource/v2/IntegrationsResourceTest.java b/api/src/test/java/io/kaoto/backend/api/resource/v2/IntegrationsResourceTest.java
@@ -413,6 +413,7 @@ void uniqueName() throws IOException {
                 .entrySet();
         assertEquals(flows.flows().size(), ids.size());
         assertTrue(ids.stream().allMatch(id -> id.getValue() == 1l));
+            assertTrue(ids.stream().allMatch(id ->Pattern.matches( "([A-Za-z0-9])+", id.getKey().toString())));
 
         //Let's assign the same name to all flows
         var sameIdentifier = "sameIdentifier";