From 474561600e4e04cf112e367e1b3e1c1b8937a956 Mon Sep 17 00:00:00 2001
From: James Salter <iteration@gmail.com>
Date: Tue, 27 Jul 2021 13:29:56 +0100
Subject: [PATCH] Fix hash == ""

---
 src/PosthogAnalytics.ts       | 23 ++++++++++++++---------
 test/PosthogAnalytics-test.ts |  6 ++++++
 2 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/src/PosthogAnalytics.ts b/src/PosthogAnalytics.ts
index 535781cb08..cdb23e582c 100644
--- a/src/PosthogAnalytics.ts
+++ b/src/PosthogAnalytics.ts
@@ -63,17 +63,22 @@ export async function getRedactedCurrentLocation(origin: string, hash: string, p
         pathname = "/<redacted_file_scheme_url>/";
     }
 
-    let [_, screen, ...parts] = hash.split("/");
+    let hashStr;
+    if (hash == "") {
+        hashStr = "";
+    } else {
+        let [_, screen, ...parts] = hash.split("/");
 
-    if (!knownScreens.has(screen)) {
-        screen = "<redacted_screen_name>";
+        if (!knownScreens.has(screen)) {
+            screen = "<redacted_screen_name>";
+        }
+
+        for (let i = 0; i < parts.length; i++) {
+            parts[i] = anonymity === Anonymity.Anonymous ? `<redacted>` : await hashHex(parts[i]);
+        }
+
+        hashStr = `${_}/${screen}/${parts.join("/")}`;
     }
-
-    for (let i = 0; i < parts.length; i++) {
-        parts[i] = anonymity === Anonymity.Anonymous ? `<redacted>` : await hashHex(parts[i]);
-    }
-
-    const hashStr = `${_}/${screen}/${parts.join("/")}`;
     return origin + pathname + hashStr;
 }
 
diff --git a/test/PosthogAnalytics-test.ts b/test/PosthogAnalytics-test.ts
index cefaafe78f..7d81b6e86d 100644
--- a/test/PosthogAnalytics-test.ts
+++ b/test/PosthogAnalytics-test.ts
@@ -174,6 +174,12 @@ bd75b3e080945674c0351f75e0db33d1e90986fa07b318ea7edf776f5eef38d4`);
             expect(location).toBe("https://foo.bar/#/<redacted_screen_name>/<redacted>/<redacted>");
         });
 
+        it("Should currently handle an empty hash", async () => {
+            const location = await getRedactedCurrentLocation(
+                "https://foo.bar", "", "/", Anonymity.Anonymous);
+            expect(location).toBe("https://foo.bar/");
+        });
+
         it("Should identify the user to posthog if pseudonymous", async () => {
             analytics.init(Anonymity.Pseudonymous);
             await analytics.identifyUser("foo");