Merge pull request #9450 from Expensify/john-hash-url-internal

Do not append & to internal Expenisfy URLs that have a # symbol.
Expensify · Jun 20, 2022 · 3546820 · 3546820
2 parents 67a022a + a0cf4b2
commit 3546820
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/libs/actions/Link.js b/src/libs/actions/Link.js
@@ -39,7 +39,11 @@ function openOldDotLink(url) {
     }
 
     function buildOldDotURL({shortLivedAuthToken}) {
-        return `${CONFIG.EXPENSIFY.EXPENSIFY_URL}${url}${url.indexOf('?') === -1 ? '?' : '&'}authToken=${shortLivedAuthToken}&email=${encodeURIComponent(currentUserEmail)}`;
+        const hasHashParams = url.indexOf('#') !== -1;
+        const hasURLParams = url.indexOf('?') !== -1;
+
+        // If the URL contains # or ?, we can assume they don't need to have the `?` token to start listing url parameters.
+        return `${CONFIG.EXPENSIFY.EXPENSIFY_URL}${url}${hasHashParams || hasURLParams ? '&' : '?'}authToken=${shortLivedAuthToken}&email=${encodeURIComponent(currentUserEmail)}`;
     }
 
     asyncOpenURL(DeprecatedAPI.GetShortLivedAuthToken(), buildOldDotURL);