ISOM-951: Better cloudmersive logging (#1323)

* fix: better logging for cloudmersive errors * fix: better logging for cloudmersive success
isomerpages · Apr 19, 2024 · 6479814 · 6479814
1 parent 4e6bc5e
commit 6479814
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/src/services/fileServices/MdPageServices/MediaFileService.js b/src/services/fileServices/MdPageServices/MediaFileService.js
@@ -36,7 +36,12 @@ class MediaFileService {
     const cmConfig = isCloudmersiveEnabled(sessionData.growthbook)
     if (cmConfig.is_enabled) {
       const virusScanRes = await scanFileForVirus(fileBuffer, cmConfig.timeout)
-      logger.info(`File scan result: ${virusScanRes.CleanResult}`)
+      logger.info({
+        message: "File scan result",
+        meta: {
+          virusScanRes,
+        },
+      })
       if (!virusScanRes || !virusScanRes.CleanResult) {
         throw new BadRequestError("File did not pass virus scan")
       }

diff --git a/src/utils/file-upload-utils.js b/src/utils/file-upload-utils.js
@@ -44,11 +44,16 @@ const scanFileForVirus = (fileBuffer, timeout) => {
     defaultCloudmersiveClient.timeout = timeout
   }
   return new Promise((success, failure) => {
-    apiInstance.scanFile(fileBuffer, (error, data) => {
+    apiInstance.scanFile(fileBuffer, (error, data, response) => {
       if (error) {
-        logger.error(
-          `Error when calling Cloudmersive Virus Scan API: ${error.message}`
-        )
+        logger.error({
+          message: "Error when calling Cloudmersive Virus Scan API",
+          error,
+          meta: {
+            data,
+            headers: response?.headers,
+          },
+        })
         failure(error)
       } else {
         logger.info("Cloudmersive Virus Scan API called successfully")