@vulkano/core 1.17.2 → 1.17.4

package/bootstrap/express.js

@@ -1,3 +1,5 @@
+const path = require('path');
+
 const merge = require('../libs/Merge');
 
 module.exports = function getExpressConfiguration() {
@@ -49,7 +51,7 @@ module.exports = function getExpressConfiguration() {
     sockets: {},
     redis: {},
     multer: {
-      dest: 'public/files'
+      dest: global.PUBLIC_PATH ? path.join(global.PUBLIC_PATH, 'files') : 'public/files'
     },
     morgan: {
       format: 'dev',

package/bootstrap/server.js

@@ -190,9 +190,11 @@ module.exports = function loadServer() {
       vulkano.use(timeout( expressConfig.timeout || 120000 ));
 
       vulkano.use( (req, res, next) => {
-        if (!req.timedout) {
-          next();
+        if (req.timedout) {
+          res.status(503).json({ success: false, statusCode: 503, error: { detail: 'Request timeout' } });
+          return;
         }
+        next();
       });
 
       // ---------------
@@ -660,6 +662,12 @@ module.exports = function loadServer() {
           return;
         }
 
+        // Timeout — always respond with JSON regardless of request type
+        if (req.timedout || (err && err.timeout)) {
+          res.status(503).json({ success: false, statusCode: 503, error: { detail: 'Request timeout' } });
+          return;
+        }
+
         const status = err ? (err.status || 500) : (res.statusCode || 500);
 
         res.status(status);
@@ -871,6 +879,9 @@ module.exports = function loadServer() {
           (pubClient ? pubClient.connect() : null),
           (subClient ? subClient.connect() : null)
         ])
+        .catch((err) => {
+          throw new Error(`Socket Redis adapter failed to connect: ${err.message}`);
+        })
         .then(() => {
 
           io.on('connection', (socket) => {

package/database/scaffold.js

@@ -97,10 +97,12 @@ module.exports = {
       });
     }
 
+    // Two queries by design: fetch the full document first so subdocument arrays
+    // and nested fields are preserved in the write — a bare $set would drop
+    // any subdoc entries not included in the incoming payload.
     return this.getByField(_id)
       .then( (record) => {
 
-        // Merge current info with incoming values
         const merged = {
           ...record,
           ...filtered,

package/libs/Jwt.js

@@ -122,12 +122,16 @@ module.exports = {
    */
   decode(token, customKey) {
 
+    if (!token) {
+      return null;
+    }
+
     const {
       key,
       expiration: allowExpiration
     } = this.getConfig();
 
-    let data = {};
+    let data = null;
 
     try {
 
@@ -135,12 +139,16 @@ module.exports = {
       data = this.decrypt(payload);
 
     } catch (e) {
-      console.log('Invalid Token');
+      // invalid token
+    }
+
+    if (!data) {
+      return null;
     }
 
     const {
       expiration
-    } = data || {};
+    } = data;
 
     const now = String(Date.now());
 
@@ -149,14 +157,13 @@ module.exports = {
       return null;
     }
 
-    // Ignore expiration field
+    // Ignore expiration requirement
     if (allowExpiration === false) {
       return data;
     }
 
-    // Expiration must be required
+    // Expiration field is required
     if (!expiration) {
-      console.log('JWT Without expiration date');
       return null;
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vulkano/core",
-  "version": "1.17.2",
+  "version": "1.17.4",
   "description": "A MVC framework using Express 4",
   "license": "MIT",
   "author": "Vulkano Team",