cluster2 - npm Package Compare versions

Comparing version 0.4.25 to 0.4.26

lib/misc.js

@@ -188,2 +188,19 @@ /*
 
+                            function stopWorker(signal){
+
+                                if(!exports.safeKill(pid, signal, logger)){
+                                    //worker still there, should emit 'exit' eventually
+                                    logger.debug('[deathQueue] worker:%d did not report death by:%d, kill by '+signal, pid, wait);
+                                    if (signal === 'SIGTERM') {
+                                        deathGuard = setTimeout(stopWorker.bind(null, 'SIGKILL'), wait);
+                                    }
+                                }
+                                else{//suicide or accident already happended, process has run away
+                                    //we emit this from master on behalf of the run away process.
+                                    logger.debug('[deathQueue] worker:%d probably ran away, emit:%s on behalf', death);
+                                    //immediately report death to the master
+                                    emitter.emit('died', pid);
+                                }
+                            }
+
                             var onDeath = function onDeath(dismiss){
@@ -213,17 +230,4 @@
                                 },
-                                deathGuard = setTimeout(function(){
+                                deathGuard = setTimeout(stopWorker.bind(null, 'SIGTERM'), wait);
 
-                                    if(!exports.safeKill(pid, 'SIGTERM', logger)){
-                                        //worker still there, should emit 'exit' eventually
-                                        logger.debug('[deathQueue] worker:%d did not report death by:%d, kill by SIGTERM', pid, wait);
-                                    }
-                                    else{//suicide or accident already happended, process has run away
-                                        //we emit this from master on behalf of the run away process.
-                                        logger.debug('[deathQueue] worker:%d probably ran away, emit:%s on behalf', death);
-                                        //immediately report death to the master
-                                        emitter.emit('died', pid);
-                                    }
-
-                                }, wait);
-
                             worker.kill('SIGINT');
@@ -251,3 +255,3 @@
 exports.deathQueue = exports.deathQueueGenerator({
-    'timeout': 60000
+    'timeout': process.env.DEATH_TIMEOUT || 60000
 });

package.json

 {
-    "author": "ql.io",
-    "contributors": [{
-        "name": "Subbu Allamaraju",
-        "email": "[e-mail]"
-    }, 
+  "author": "ql.io",
+  "contributors": [
     {
-        "name": "Roy Zhou",
-        "email": "[e-mail]"
-    }],
-    "name": "cluster2",
-    "version": "0.4.25",
-    "repository": {
-        "type": "git",
-        "url": "https://github.com/cubejs/cluster2"
+      "name": "Subbu Allamaraju",
+      "email": "[e-mail]"
     },
-    "engines": {
-        "node": ">= 0.8.0"
-    },
-    "main": "lib/index.js",
-    "dependencies": {
-        "underscore": "~1.4.4",
-        "express": "~2.5.11",
-        "ejs": "~0.8.4",
-        "npm": "~1.3.0",
-        "when": "~2.4.0",
-        "usage": "~0.3.8",
-        "bignumber.js": "~1.1.1",
-        "gc-stats": "~0.0.1"
-    },
-    "devDependencies": {
-        "websocket": "~1.0.8",
-        "nodeunit": "~0.8.0",
-        "request": "~2.21.0",
-        "mocha": "~1.11.0",
-        "should": "~1.2.2",
-        "harbor": "~0.2.0"
-    },
-    "scripts": {
-        "test": "nodeunit test"
-    },
-    "optionalDependencies": {},
-    "publishConfig": {
-        "registry": "https://registry.npmjs.org/"
+    {
+      "name": "Roy Zhou",
+      "email": "[e-mail]"
     }
-}
+  ],
+  "name": "cluster2",
+  "version": "0.4.26",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/cubejs/cluster2"
+  },
+  "engines": {
+    "node": ">= 0.8.0"
+  },
+  "main": "lib/index.js",
+  "dependencies": {
+    "underscore": "~1.4.4",
+    "express": "~2.5.11",
+    "ejs": "~0.8.4",
+    "npm": "~1.3.0",
+    "when": "~2.4.0",
+    "usage": "~0.3.8",
+    "bignumber.js": "~1.1.1",
+    "gc-stats": "~0.0.1"
+  },
+  "devDependencies": {
+    "websocket": "~1.0.8",
+    "nodeunit": "~0.8.0",
+    "request": "~2.21.0",
+    "mocha": "~1.11.0",
+    "should": "~1.2.2",
+    "harbor": "~0.2.0"
+  },
+  "scripts": {
+    "test": "nodeunit test"
+  },
+  "optionalDependencies": {},
+  "publishConfig": {
+    "registry": "https://registry.npmjs.org/"
+  },
+  "ebay": {}
+}

New alerts

Environment variable access

Supply chain risk

Package accesses environment variables, which may be a sign of credential stuffing or data theft.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

120852

increased by0.13%

Lines of code

2036

increased by0.2%

Worsened metrics

Number of package files

26

decreased by-3.7%

Number of low supply chain risk alerts

6

increased by20%

No dependency changes