Skip to content

Commit

Permalink
cluster: fix cluster rr distribute error
Browse files Browse the repository at this point in the history
PR-URL: nodejs#44202
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
theanarkh authored and Fyko committed Sep 15, 2022
1 parent 797cb13 commit 9c457b2
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 0 deletions.
4 changes: 4 additions & 0 deletions lib/internal/cluster/round_robin_handle.js
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,10 @@ RoundRobinHandle.prototype.remove = function(worker) {
};

RoundRobinHandle.prototype.distribute = function(err, handle) {
// If `accept` fails just skip it (handle is undefined)
if (err) {
return;
}
append(this.handles, handle);
// eslint-disable-next-line node-core/no-array-destructuring
const [ workerEntry ] = this.free; // this.free is a SafeMap
Expand Down
30 changes: 30 additions & 0 deletions test/parallel/test-cluster-accept-fail.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
// Flags: --expose-internals
'use strict';
const common = require('../common');
const assert = require('assert');
const net = require('net');
const cluster = require('cluster');
const rr = require('internal/cluster/round_robin_handle');

if (cluster.isPrimary) {
const distribute = rr.prototype.distribute;
rr.prototype.distribute = function(err, handle) {
assert.strictEqual(err, 0);
handle.close();
distribute.call(this, -1, undefined);
};
cluster.schedulingPolicy = cluster.SCHED_RR;
cluster.fork();
} else {
const server = net.createServer(common.mustNotCall());
server.listen(0, common.mustCall(() => {

const socket = net.connect(server.address().port);

socket.on('close', common.mustCall(() => {
server.close(common.mustCall(() => {
process.disconnect();
}));
}));
}));
}

0 comments on commit 9c457b2

Please sign in to comment.