feat: log href without including search params

juspay · Jun 12, 2024 · 6489aa6 · 6489aa6
1 parent a3526d2
commit 6489aa6
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 7 deletions.
diff --git a/src/LoaderController.res b/src/LoaderController.res
@@ -270,7 +270,7 @@ let make = (~children, ~paymentMode, ~setIntegrateErrorError, ~logger, ~initTime
               setLaunchTime(_ => newLaunchTime)
               let initLoadlatency = Date.now() -. newLaunchTime
               logger.setLogInfo(
-                ~value=Window.href,
+                ~value=Window.hrefWithoutSearch,
                 ~eventName=APP_RENDERED,
                 ~latency=initLoadlatency,
                 (),

diff --git a/src/Window.res b/src/Window.res
@@ -122,11 +122,14 @@ external sendBeacon: (string, string) => unit = "sendBeacon"
 external hostname: string = "hostname"
 
 @val @scope(("window", "location"))
-external href: string = "href"
+external origin: string = "origin"
 
 @val @scope(("window", "location"))
 external protocol: string = "protocol"
 
+@val @scope(("window", "location"))
+external pathname: string = "pathname"
+
 let isSandbox = hostname === "beta.hyperswitch.io"
 
 let isInteg = hostname === "dev.hyperswitch.io"
@@ -140,3 +143,5 @@ module Location = {
 module Element = {
   @get external clientWidth: Dom.element => int = "clientWidth"
 }
+
+let hrefWithoutSearch = origin ++ pathname
diff --git a/src/orca-loader/Hyper.res b/src/orca-loader/Hyper.res
@@ -128,7 +128,12 @@ let make = (publishableKey, options: option<JSON.t>, analyticsInfo: option<JSON.
       () => {
         logger.setMerchantId(publishableKey)
         logger.setSessionId(sessionID)
-        logger.setLogInfo(~value=Window.href, ~eventName=APP_INITIATED, ~timestamp=sdkTimestamp, ())
+        logger.setLogInfo(
+          ~value=Window.hrefWithoutSearch,
+          ~eventName=APP_INITIATED,
+          ~timestamp=sdkTimestamp,
+          (),
+        )
       }
     }->Sentry.sentryLogger
     let isSecure = Window.protocol === "https:"
@@ -137,8 +142,8 @@ let make = (publishableKey, options: option<JSON.t>, analyticsInfo: option<JSON.
       ->Array.find(url => Window.hostname->String.includes(url))
       ->Option.isSome
     if !isSecure && !isLocal {
-      manageErrorWarning(HTTP_NOT_ALLOWED, ~dynamicStr=Window.href, ~logger, ())
-      Exn.raiseError("Insecure domain: " ++ Window.href)
+      manageErrorWarning(HTTP_NOT_ALLOWED, ~dynamicStr=Window.hrefWithoutSearch, ~logger, ())
+      Exn.raiseError("Insecure domain: " ++ Window.hrefWithoutSearch)
     }
     switch Window.getHyper->Nullable.toOption {
     | Some(hyperMethod) => {
@@ -404,7 +409,7 @@ let make = (publishableKey, options: option<JSON.t>, analyticsInfo: option<JSON.
           resolve(JSON.Encode.null)
         })
         ->then(_ => {
-          logger.setLogInfo(~value=Window.href, ~eventName=ORCA_ELEMENTS_CALLED, ())
+          logger.setLogInfo(~value=Window.hrefWithoutSearch, ~eventName=ORCA_ELEMENTS_CALLED, ())
           resolve()
         })
         ->ignore
@@ -544,7 +549,11 @@ let make = (publishableKey, options: option<JSON.t>, analyticsInfo: option<JSON.
           resolve(JSON.Encode.null)
         })
         ->then(_ => {
-          logger.setLogInfo(~value=Window.href, ~eventName=PAYMENT_SESSION_INITIATED, ())
+          logger.setLogInfo(
+            ~value=Window.hrefWithoutSearch,
+            ~eventName=PAYMENT_SESSION_INITIATED,
+            (),
+          )
           resolve()
         })
         ->ignore