fix: path fetching for home paths Urls (#448)

hypertrace · Feb 5, 2024 · ed7b189 · ed7b189
1 parent 475aca8
commit ed7b189
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 6 deletions.
diff --git a/.../main/java/org/hypertrace/semantic/convention/utils/http/HttpSemanticConventionUtils.java b/.../main/java/org/hypertrace/semantic/convention/utils/http/HttpSemanticConventionUtils.java
@@ -438,7 +438,7 @@ public static boolean isValidUrl(String url) {
   }
 
   public static URL getNormalizedUrl(String url) throws MalformedURLException, URISyntaxException {
-    String sanitizedUrl = StringUtils.stripStart(url, SLASH);
+    String sanitizedUrl = removeInitialSlashes(url);
     URL absoluteUrl = new URL(new URL(RELATIVE_URL_CONTEXT), sanitizedUrl);
     return absoluteUrl.toURI().normalize().toURL();
   }
@@ -510,7 +510,7 @@ public static Optional<String> getHttpPath(Event event) {
         if (StringUtils.isBlank(pathVal)) {
           pathVal = SLASH;
         }
-        return Optional.of(removeTrailingSlash(pathVal));
+        return Optional.of(removeTrailingSlashes(pathVal));
       } catch (MalformedURLException | URISyntaxException e) {
         LOGGER.debug(
             "On extracting httpPath, received an invalid URL: {}, {}", url.get(), e.getMessage());
@@ -530,7 +530,7 @@ private static Optional<String> getHttpPathFromRawAttributes(Event event) {
       if (attributeValueMap.get(path) != null) {
         String s = attributeValueMap.get(path).getValue();
         if (StringUtils.isNotBlank(s) && s.startsWith(SLASH)) {
-          return getPathFromUrlObject(s).map(HttpSemanticConventionUtils::removeTrailingSlash);
+          return getPathFromUrlObject(s).map(HttpSemanticConventionUtils::removeTrailingSlashes);
         }
       }
     }
@@ -897,9 +897,22 @@ static Optional<String> getPathFromUrlObject(String urlPath) {
     return Optional.empty();
   }
 
-  private static String removeTrailingSlash(String s) {
-    // Ends with "/" and it's not home page path
-    return s.endsWith(SLASH) && s.length() > 1 ? s.substring(0, s.length() - 1) : s;
+  private static String removeTrailingSlashes(String url) {
+    // if it's home page path, then return /
+    String updatedUrl = StringUtils.stripEnd(url, SLASH);
+    if (updatedUrl.isEmpty()) {
+      return SLASH;
+    }
+    return updatedUrl;
+  }
+
+  private static String removeInitialSlashes(String url) {
+    // if it's home page path, then return /
+    String updatedUrl = StringUtils.stripStart(url, SLASH);
+    if (updatedUrl.isEmpty()) {
+      return SLASH;
+    }
+    return updatedUrl;
   }
 
   @Nullable

diff --git a/...t/java/org/hypertrace/semantic/convention/utils/http/HttpSemanticConventionUtilsTest.java b/...t/java/org/hypertrace/semantic/convention/utils/http/HttpSemanticConventionUtilsTest.java
@@ -665,4 +665,20 @@ public void testGetPrimaryDomain() {
 
     Assertions.assertEquals(HttpSemanticConventionUtils.getPrimaryDomain("10.0.0.0"), "10.0.0.0");
   }
+
+  @Test
+  void testGetPathFromUrlObject() {
+    Optional<String> path =
+        HttpSemanticConventionUtils.getPathFromUrlObject("http://app.test.com:9191/");
+    Assertions.assertTrue(path.isPresent());
+    Assertions.assertEquals("/", path.get());
+
+    path = HttpSemanticConventionUtils.getPathFromUrlObject("http://app.test.com:9191//");
+    Assertions.assertTrue(path.isPresent());
+    Assertions.assertEquals("/", path.get());
+
+    path = HttpSemanticConventionUtils.getPathFromUrlObject("http://app.test.com:9191//abc/def");
+    Assertions.assertTrue(path.isPresent());
+    Assertions.assertEquals("/abc/def", path.get());
+  }
 }