fix: account urls are not per project (#19933)

* fix: account urls are not per project * add a test or two
PostHog · Jan 23, 2024 · 8cbd8d4 · 8cbd8d4
1 parent 3c8627a
commit 8cbd8d4
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/frontend/src/lib/utils/router-utils.test.ts b/frontend/src/lib/utils/router-utils.test.ts
@@ -0,0 +1,12 @@
+import { addProjectIdIfMissing } from 'lib/utils/router-utils'
+
+describe('router-utils', () => {
+    it('does not redirect account URLs to a project URL', () => {
+        const altered = addProjectIdIfMissing('/account/two_factor', 123)
+        expect(altered).toEqual('/account/two_factor')
+    })
+    it('does not allow account urls to have a project url', () => {
+        const altered = addProjectIdIfMissing('/project/123/account/two_factor', 123)
+        expect(altered).toEqual('/account/two_factor')
+    })
+})
diff --git a/frontend/src/lib/utils/router-utils.ts b/frontend/src/lib/utils/router-utils.ts
@@ -11,6 +11,7 @@ const pathsWithoutProjectId = [
     'login',
     'signup',
     'create-organization',
+    'account',
 ]
 
 function isPathWithoutProjectId(path: string): boolean {