Bug 1472107 - Handle URL error in getUrlQuery. r?odvarko draft
authorPavan Veginati <pavanon9@gmail.com>
Wed, 04 Jul 2018 23:16:49 +0530
changeset 814189 eebbf27b2eeeccc441cb5b1a1712058201745e17
parent 814112 40423bb8aa180813c4eb1cc58dcee9f0ad440841
push id115125
push userbmo:pavanon9@gmail.com
push dateWed, 04 Jul 2018 18:04:48 +0000
reviewersodvarko
bugs1472107
milestone63.0a1
Bug 1472107 - Handle URL error in getUrlQuery. r?odvarko MozReview-Commit-ID: KHbwA1FSVyg
devtools/client/netmonitor/src/utils/request-utils.js
--- a/devtools/client/netmonitor/src/utils/request-utils.js
+++ b/devtools/client/netmonitor/src/utils/request-utils.js
@@ -168,17 +168,21 @@ function getUrlBaseName(url) {
 
 /**
  * Helpers for getting the query portion of a url.
  *
  * @param {string} url - url string
  * @return {string} unicode query of a url
  */
 function getUrlQuery(url) {
-  return (new URL(url)).search.replace(/^\?/, "");
+  try {
+     return (new URL(url)).search.replace(/^\?/, "");
+  } catch (err) {
+     return null;
+  }
 }
 
 /**
  * Helpers for getting unicode name and query portions of a url.
  *
  * @param {string} url - url string
  * @return {string} unicode basename and query portions of a url
  */