chore(hb_parse_error): created a JSON validation function to prevent …

…HB error and correctly log errors (#18199)
influxdata · May 22, 2020 · 24ce360 · 24ce360
1 parent 083f72a
commit 24ce360
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/ui/src/localStorage.ts b/ui/src/localStorage.ts
@@ -29,13 +29,23 @@ export const loadLocalStorage = (): LocalStorage => {
   }
 }
 
+const isValidJSONString = errorString => {
+  try {
+    JSON.parse(errorString)
+  } catch (e) {
+    return false
+  }
+  return true
+}
+
 export const saveToLocalStorage = (state: LocalStorage): void => {
   try {
     window.localStorage.setItem(
       'state',
       JSON.stringify(normalizeSetLocalStorage(state))
     )
-  } catch (err) {
-    console.error('Unable to save state to local storage: ', JSON.parse(err))
+  } catch (error) {
+    const errorMessage = isValidJSONString(error) ? JSON.parse(error) : error
+    console.error('Unable to save state to local storage: ', errorMessage)
   }
 }