@jsreport/jsreport-core 3.3.0 → 3.4.0

package/README.md

@@ -282,6 +282,13 @@ jsreport.documentStore.collection('templates')
 
 ## Changelog
 
+### 3.4.0
+
+- fix for reports execution
+- fix for render profiling
+- fix for blob storage remove
+- update deps to fix npm audit
+
 ### 3.1.0
 
 - fix blob storage append to not existing blob (mongo)

package/lib/main/optionsLoad.js

@@ -133,15 +133,9 @@ async function loadConfig (defaults, options, loadExternal = true) {
         const currentValue = obj.value[extensionKey]
         delete obj.value[extensionKey]
 
-        if (realExtensionName !== extensionKey && obj.value[realExtensionName]) {
-          obj.value[realExtensionName] = extend(
-            true,
-            obj.value[realExtensionName],
-            currentValue
-          )
-        } else {
-          obj.value[realExtensionName] = currentValue
-        }
+        // the camelCase key version should already contain all merged values
+        // (from both the real extension name with "-" and camel case)
+        obj.value[realExtensionName] = currentValue
       })
     } else if (!normalize && obj.key.startsWith('extensions')) {
       // the transform ensures that camelCase alias keys of extensions

package/lib/main/reporter.js

@@ -405,6 +405,7 @@ class MainReporter extends Reporter {
       if (req.context.returnResponseAndKeepWorker) {
         keepWorker = true
         res.stream = Readable.from(res.content)
+        await this.afterRenderListeners.fire(req, res)
         return res
       }
 

package/lib/worker/render/profiler.js

@@ -8,7 +8,7 @@ class Profiler {
   constructor (reporter) {
     this.reporter = reporter
 
-    this.reporter.addRequestContextMetaConfig('profiling', { sandboxReadOnly: true })
+    this.reporter.addRequestContextMetaConfig('profiling', { sandboxHidden: true })
     this.reporter.addRequestContextMetaConfig('resolvedTemplate', { sandboxHidden: true })
 
     this.reporter.beforeMainActionListeners.add('profiler', (actionName, data, req) => {
@@ -93,7 +93,7 @@ class Profiler {
 
       m.req = { diff: createPatch('req', req.context.profiling.reqLastVal || '', stringifiedReq, 0) }
 
-      req.context.profiling.resLastVal = res.content
+      req.context.profiling.resLastVal = (res.content == null || isbinaryfile(res.content)) ? null : res.content.toString()
       req.context.profiling.resMetaLastVal = stringifiedResMeta
       req.context.profiling.reqLastVal = stringifiedReq
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsreport/jsreport-core",
-  "version": "3.3.0",
+  "version": "3.4.0",
   "description": "javascript based business reporting",
   "keywords": [
     "report",
@@ -53,8 +53,8 @@
     "lodash.set": "4.3.2",
     "lru-cache": "4.1.1",
     "ms": "2.1.3",
-    "nanoid": "3.1.16",
-    "nconf": "0.10.0",
+    "nanoid": "3.2.0",
+    "nconf": "0.11.3",
     "node.extend.without.arrays": "1.1.6",
     "reap2": "1.0.1",
     "semver": "7.3.5",

package/test/blobStorage/common.js

@@ -18,4 +18,8 @@ module.exports = (storage) => {
     const buf = await storage().read('foldera/folderb/myblob.txt')
     buf.toString().should.be.eql('hula')
   })
+
+  it('remove shouldnt fail for missing blob', async () => {
+    await storage().remove('foo')
+  })
 }