From d974db99ceec505098cb46861fa3b685771a1661 Mon Sep 17 00:00:00 2001
From: Umesh Kumar <166806589+TangoBeeAkto@users.noreply.github.com>
Date: Mon, 10 Feb 2025 17:42:32 +0530
Subject: [PATCH] clean code

---
 .../src/main/java/com/akto/runtime/APICatalogSync.java   | 4 ++--
 .../java/com/akto/hybrid_runtime/APICatalogSync.java     | 4 ++--
 .../src/main/java/com/akto/dto/HttpResponseParams.java   | 9 +++++++++
 .../dao/src/main/java/com/akto/dto/type/URLTemplate.java | 3 ++-
 .../src/main/java/com/akto/graphql/GraphQLUtils.java     | 7 -------
 5 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/apps/api-runtime/src/main/java/com/akto/runtime/APICatalogSync.java b/apps/api-runtime/src/main/java/com/akto/runtime/APICatalogSync.java
index 51c4c7967b..d611be5bd5 100644
--- a/apps/api-runtime/src/main/java/com/akto/runtime/APICatalogSync.java
+++ b/apps/api-runtime/src/main/java/com/akto/runtime/APICatalogSync.java
@@ -741,7 +741,7 @@ public static URLTemplate tryParamteresingUrl(URLStatic newUrl){
 
         int start = newUrl.getUrl().startsWith("http") ? 3 : 0;
 
-        if(GraphQLUtils.isGraphQLEndpoint(newUrl.getUrl())) {
+        if(HttpResponseParams.isGraphQLEndpoint(newUrl.getUrl())) {
             return null; // Don't merge GraphQL endpoints
         }
 
@@ -811,7 +811,7 @@ public static URLTemplate tryMergeUrls(URLStatic dbUrl, URLStatic newUrl) {
         SuperType[] newTypes = new SuperType[newTokens.length];
         int templatizedStrTokens = 0;
 
-        if(GraphQLUtils.isGraphQLEndpoint(dbUrl.getUrl()) || GraphQLUtils.isGraphQLEndpoint(newUrl.getUrl())) {
+        if(HttpResponseParams.isGraphQLEndpoint(dbUrl.getUrl()) || HttpResponseParams.isGraphQLEndpoint(newUrl.getUrl())) {
             return null; // Don't merge GraphQL endpoints
         }
 
diff --git a/apps/mini-runtime/src/main/java/com/akto/hybrid_runtime/APICatalogSync.java b/apps/mini-runtime/src/main/java/com/akto/hybrid_runtime/APICatalogSync.java
index e871cc65a6..45c352055e 100644
--- a/apps/mini-runtime/src/main/java/com/akto/hybrid_runtime/APICatalogSync.java
+++ b/apps/mini-runtime/src/main/java/com/akto/hybrid_runtime/APICatalogSync.java
@@ -527,7 +527,7 @@ public static URLTemplate tryParamteresingUrl(URLStatic newUrl){
 
         int start = newUrl.getUrl().startsWith("http") ? 3 : 0;
 
-        if(GraphQLUtils.isGraphQLEndpoint(newUrl.getUrl())) {
+        if(HttpResponseParams.isGraphQLEndpoint(newUrl.getUrl())) {
             return null; // Don't merge GraphQL endpoints
         }
 
@@ -597,7 +597,7 @@ public static URLTemplate tryMergeUrls(URLStatic dbUrl, URLStatic newUrl) {
         SuperType[] newTypes = new SuperType[newTokens.length];
         int templatizedStrTokens = 0;
 
-        if(GraphQLUtils.isGraphQLEndpoint(dbUrl.getUrl()) || GraphQLUtils.isGraphQLEndpoint(newUrl.getUrl())) {
+        if(HttpResponseParams.isGraphQLEndpoint(dbUrl.getUrl()) || HttpResponseParams.isGraphQLEndpoint(newUrl.getUrl())) {
             return null; // Don't merge GraphQL endpoints
         }
 
diff --git a/libs/dao/src/main/java/com/akto/dto/HttpResponseParams.java b/libs/dao/src/main/java/com/akto/dto/HttpResponseParams.java
index 7fb7ed7e5b..235645579c 100644
--- a/libs/dao/src/main/java/com/akto/dto/HttpResponseParams.java
+++ b/libs/dao/src/main/java/com/akto/dto/HttpResponseParams.java
@@ -98,6 +98,15 @@ public static boolean isGraphql(HttpResponseParams responseParams) {
         return isAllowedForParse && requestPayload.contains(QUERY);
     }
 
+    public static boolean isGraphQLEndpoint(String url) {
+        for (String keyword : allowedPath) {
+            if (url.contains(keyword)) {
+                return true;
+            }
+        }
+        return false;
+    }
+
     public int getTimeOrNow() {
         return getTime() == 0 ? Context.now() : getTime();
     }
diff --git a/libs/dao/src/main/java/com/akto/dto/type/URLTemplate.java b/libs/dao/src/main/java/com/akto/dto/type/URLTemplate.java
index ade8a9eaee..456d5f937a 100644
--- a/libs/dao/src/main/java/com/akto/dto/type/URLTemplate.java
+++ b/libs/dao/src/main/java/com/akto/dto/type/URLTemplate.java
@@ -4,6 +4,7 @@
 import java.util.UUID;
 
 import com.akto.dao.context.Context;
+import com.akto.dto.HttpResponseParams;
 import com.akto.dto.type.SingleTypeInfo.SuperType;
 import com.akto.dto.type.URLMethods.Method;
 
@@ -66,7 +67,7 @@ public boolean match(String[] url, Method urlMethod) {
         String[] thatTokens = url;
         if (thatTokens.length != this.tokens.length) return false;
 
-        if(Arrays.toString(url).contains("graphql") || Arrays.toString(url).contains("graph")) {
+        if(HttpResponseParams.isGraphQLEndpoint(Arrays.toString(url))) {
             return false;
         }
 
diff --git a/libs/utils/src/main/java/com/akto/graphql/GraphQLUtils.java b/libs/utils/src/main/java/com/akto/graphql/GraphQLUtils.java
index ded895dce9..5a742b13a4 100644
--- a/libs/utils/src/main/java/com/akto/graphql/GraphQLUtils.java
+++ b/libs/utils/src/main/java/com/akto/graphql/GraphQLUtils.java
@@ -320,11 +320,4 @@ public List<OperationDefinition> parseGraphQLRequest(Map requestPayload) {
         }
         return result;
     }
-
-    public static boolean isGraphQLEndpoint(String url) {
-        if(url.contains("graphql") || url.contains("graph")) {
-            return true;
-        }
-        return false;
-    }
 }