mythix 2.4.7 → 2.4.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mythix",
-  "version": "2.4.7",
+  "version": "2.4.10",
   "description": "Mythix is a NodeJS web-app framework",
   "main": "src/index.js",
   "scripts": {
@@ -30,7 +30,7 @@
     "express": "^4.17.3",
     "express-busboy": "github:th317erd/express-busboy#0754a570d7979097b31e48655b80d3fcd628d4e4",
     "form-data": "^4.0.0",
-    "mythix-orm": "^1.5.1",
+    "mythix-orm": "^1.5.2",
     "nife": "^1.11.3",
     "prompts": "^2.4.2"
   }

package/src/cli/cli-utils.js

@@ -367,8 +367,9 @@ function spawnCommand(args, options, _config) {
   });
 }
 
-async function executeCommand(configPath, applicationCommandsPath, yargsPath, simpleYargsPath, argv, commandPath, command, config) {
+async function executeCommand(configPath, applicationCommandsPath, yargsPath, simpleYargsPath, argv, commandPath, command, _config) {
   try {
+    let config        = _config || {};
     let Klass         = CommandBase.commands[command];
     let nodeArguments = ((config.runtime || 'node') === 'node') ? (Klass.nodeArguments || []) : [];
     let args          = nodeArguments.concat([ commandPath ], argv);

package/src/controllers/generate-client-api-interface.js

@@ -83,12 +83,18 @@ function nodeRequestHandler(routeName, requestOptions) {
       },
       requestOptions,
       extraConfig,
-      { headers:  Object.assign({}, headers, Utils.keysToLowerCase(extraConfig.headers || {})) },
+      {
+        headers: Utils.cleanObjectProperties(Object.assign(
+          {},
+          headers,
+          Utils.keysToLowerCase(extraConfig.headers || {})
+        )),
+      },
     );
 
     delete options.data;
 
-    var thisRequest = HTTP.request(options, function(response) {
+    var thisRequest = HTTP.request(Utils.cleanObjectProperties(options), function(response) {
       var responseData = Buffer.alloc(0);
 
       response.on('data', function(chunk) {
@@ -172,7 +178,7 @@ function browserRequestHandler(routeName, requestOptions) {
     var url         = requestOptions.url;
     var data        = requestOptions.data;
     var extraConfig = {};
-    var headers     = Object.assign({ 'content-type': 'application/json; charset=UTF-8' }, Utils.keysToLowerCase(this.defaultHeaders || {}), Utils.keysToLowerCase(requestOptions.headers || {}));
+    var headers     = Object.assign(Utils.keysToLowerCase(this.defaultHeaders || {}), Utils.keysToLowerCase(requestOptions.headers || {}));
 
     if (data) {
       if (!method.match(/^(GET|HEAD)$/i)) {
@@ -193,12 +199,18 @@ function browserRequestHandler(routeName, requestOptions) {
       { method },
       requestOptions,
       extraConfig,
-      { headers:  Object.assign({}, headers, Utils.keysToLowerCase(extraConfig.headers || {})) },
+      {
+        headers: Utils.cleanObjectProperties(Object.assign(
+          {},
+          headers,
+          Utils.keysToLowerCase(extraConfig.headers || {})
+        )),
+      },
     );
 
     delete options.data;
 
-    globalScope.fetch(url, options).then(
+    globalScope.fetch(url, Utils.cleanObjectProperties(options)).then(
       function(response) {
         if (typeof requestOptions.responseHandler === 'function')
           return requestOptions.responseHandler(response);
@@ -271,6 +283,22 @@ function generateUtils() {
       return newObj;
     }
 
+    function cleanObjectProperties(obj) {
+      var keys    = Object.keys(obj || {});
+      var newObj  = {};
+
+      for (var i = 0, il = keys.length; i < il; i++) {
+        var key   = keys[i];
+        var value = obj[key];
+        if (value == null || value == '')
+          continue;
+
+        newObj[key] = value;
+      }
+
+      return newObj;
+    }
+
     function injectURLParams(routeName, _options) {
       var options = _options || {};
       var params = options.params || {};
@@ -299,6 +327,7 @@ function generateUtils() {
       isEmpty,
       dataToQueryString,
       keysToLowerCase,
+      cleanObjectProperties,
       injectURLParams,
     };
   })();
@@ -350,11 +379,15 @@ function generateRoutes(_routes, _options) {
         contentType = contentType[0];
 
       clientOptions = {
-        credentials:  'same-origin',
-        headers:      {
-          'Content-Type': contentType,
-        },
+        credentials: 'same-origin',
       };
+
+      // Don't set content type for multipart/form-data
+      if (!(/multipart\/form-data/i).test(contentType)) {
+        clientOptions.headers = {
+          'Content-Type': contentType,
+        };
+      }
     }
 
     clientOptions = JSON.stringify(clientOptions, (key, value) => {