fix(core): Improved null check for sanitizer.

dynatrace-oss · Feb 17, 2022 · b14bbac · b14bbac
1 parent c87af35
commit b14bbac
Showing 1 changed file with 14 additions and 11 deletions.
diff --git a/libs/barista-components/core/src/util/sanitize.ts b/libs/barista-components/core/src/util/sanitize.ts
@@ -15,6 +15,7 @@
  */
 import { SecurityContext } from '@angular/core';
 import { DomSanitizer } from '@angular/platform-browser';
+import { isObject } from './type-util';
 
 /** Sanitizes a nested object or string from malicious html code  */
 export const sanitize = <T extends {} | string>(
@@ -25,16 +26,18 @@ export const sanitize = <T extends {} | string>(
     return sanitizer.sanitize(SecurityContext.HTML, option) as T;
   }
 
-  Object.keys(option).forEach((key) => {
-    if (typeof option[key] === 'string') {
-      option[key] = sanitizer.sanitize(SecurityContext.HTML, option[key]);
-    } else if (Array.isArray(option[key])) {
-      option[key].forEach((item, i) => {
-        option[key][i] = sanitize(item, sanitizer);
-      });
-    } else if (typeof option[key] === 'object') {
-      option[key] = sanitize(option[key], sanitizer);
-    }
-  });
+  if (option && isObject(option)) {
+    Object.keys(option).forEach((key) => {
+      if (typeof option[key] === 'string') {
+        option[key] = sanitizer.sanitize(SecurityContext.HTML, option[key]);
+      } else if (Array.isArray(option[key])) {
+        option[key].forEach((item, i) => {
+          option[key][i] = sanitize(item, sanitizer);
+        });
+      } else if (typeof option[key] === 'object') {
+        option[key] = sanitize(option[key], sanitizer);
+      }
+    });
+  }
   return option;
 };