@capillarytech/creatives-library 7.12.117 → 7.12.119

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@capillarytech/creatives-library",
   "author": "meharaj",
-  "version": "7.12.117",
+  "version": "7.12.119",
   "description": "Capillary creatives ui",
   "main": "./index.js",
   "module": "./index.es.js",

package/utils/cdnTransformation.js

@@ -149,7 +149,7 @@ export const getCdnUrl = ({
     let CREATIVES_S3_BUCKET_PATH = getLocalStorageItem(CREATIVES_S3_BUCKET_PATH_KEY);
     
     //remove all leading and trailing forward slash with empty string, if leading/trailing forward slash is present in node env by mistake then url will not be correct hence removing them.
-    CREATIVES_S3_BUCKET_PATH = CREATIVES_S3_BUCKET_PATH.replace(/^\/|\/$/g, '');
+    CREATIVES_S3_BUCKET_PATH = CREATIVES_S3_BUCKET_PATH?.replace(/^\/|\/$/g, '');
 
     //Basic validations on data stored in local storage.
     if (
@@ -260,6 +260,9 @@ export const getCdnUrl = ({
  * @returns htmlContents with updated urls in image tag
  */
 export const updateImagesInHtml = (htmlContents) => {
+  if(!htmlContents || typeof htmlContents !== "string"){
+    return htmlContents;
+  } 
   const copiedHtmlContents = htmlContents;
   try {
     const root = parse(htmlContents);

package/utils/tests/cdnTransformation.test.js

@@ -194,6 +194,12 @@ describe("cdnTransformationTests", () => {
       const expected = mockdata.emailRawTransformed;
       expect(htmlContentsOutput).toEqual(expected);
     });
+
+    it("should return input itself if input is either among undefined/null/empty string", () => {
+      expect(cdnUtils.updateImagesInHtml(undefined)).toEqual(undefined);
+      expect(cdnUtils.updateImagesInHtml(null)).toEqual(null);
+      expect(cdnUtils.updateImagesInHtml("")).toEqual("");
+    });
   });
 
   describe("transformEmailTemplate()", () => {