refactor new strategies

README.md

@@ -82,7 +82,10 @@ var scriptManager = require("script-manager")({
 		host: '127.0.0.1',
 		/* set a specific port range for script execution server */
 		portLeftBoundary: 1000,
-		portRightBoundary: 2000
+		portRightBoundary: 2000,
+		/* switch to use dedicated process for script evalution, this can help with
+		some issues caused by corporate proxies */
+		strategy: "http-server | dedicated-process"
 	});
 ```
 

index.js

@@ -4,7 +4,7 @@ module.exports = function(options) {
     options.timeout = options.timeout || 10000;
     options.strategy = options.strategy || "http-servers";
 
-    if (options.strategy === "http-servers") {
+    if (options.strategy === "http-server") {
         return new (require("./lib/manager-servers.js"))(options);
     }
 

lib/manager-processes.js

@@ -71,7 +71,7 @@ ScriptsManager.prototype.execute = function (inputs, options, cb) {
 
         worker.kill();
 
-        cb(new Error("Timeout error during rendering"));
+        cb(new Error("Timeout error during executing script"));
     },  options.timeout || this.options.timeout);
 };
 

lib/manager-servers.js

@@ -182,6 +182,10 @@ ScriptsManager.prototype.execute = function (inputs, options, cb) {
             return cb(err);
         }
 
+        if (!body) {
+            return cb(new Error("Something went wrong in communication with internal scripting server. You may try to change scripting strategy from `http-server` to `dedicated-process`."))
+        }
+
         if (body.error) {
             var e = new Error();
             e.message = body.error.message;