fix: handles in a SafeMap

Signed-off-by: Jérôme Benoit <jerome.benoit@sap.com>
nodejs · Oct 23, 2023 · c348acb · c348acb
1 parent ae947a3
commit c348acb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/internal/cluster/child.js b/lib/internal/cluster/child.js
@@ -270,7 +270,7 @@ function _disconnect(primaryInitiated) {
     }
   }
 
-  for (const handle of handles) {
+  for (const handle of handles.values()) {
     waitingCount++;
 
     if (handle[owner_symbol])

diff --git a/lib/internal/cluster/primary.js b/lib/internal/cluster/primary.js
@@ -153,7 +153,7 @@ function removeHandlesForWorker(worker) {
   assert(worker);
 
   // eslint-disable-next-line node-core/no-array-destructuring
-  for (const [key, handle] of handles.entries()) {
+  for (const [key, handle] of handles) {
     if (handle.remove(worker))
       handles.delete(key);
   }