Merge pull request #23774 from eh2077/21266-try-catch-regex-execution…

…-error Fix: regex execution Maximum regex stack depth reached error
Expensify · Aug 2, 2023 · b755df5 · b755df5
2 parents 15abe02 + 5066b2b
commit b755df5
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 8 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -79,7 +79,7 @@
     "date-fns-tz": "^2.0.0",
     "dom-serializer": "^0.2.2",
     "domhandler": "^4.3.0",
-    "expensify-common": "git+ssh://[email protected]/Expensify/expensify-common.git#98d8fea356f114f8b5b0cea889a41b355e5daf58",
+    "expensify-common": "git+ssh://[email protected]/Expensify/expensify-common.git#9940dd127c2d44809c98ee628a8057f08c93bfc9",
     "fbjs": "^3.0.2",
     "htmlparser2": "^7.2.0",
     "jest-when": "^3.5.2",

diff --git a/src/libs/Url.js b/src/libs/Url.js
@@ -19,7 +19,15 @@ function addTrailingForwardSlash(url) {
  */
 function getURLObject(href) {
     const urlRegex = new RegExp(URL_WEBSITE_REGEX, 'gi');
-    const match = urlRegex.exec(href);
+    let match;
+    try {
+        if (!href.startsWith('mailto:')) {
+            match = urlRegex.exec(href);
+        }
+    } catch (e) {
+        // eslint-disable-next-line no-console
+        console.warn('Error parsing url in Url.getURLObject', {error: e});
+    }
     if (!match) {
         return {
             href: undefined,