hypercore 10.29.1 → 10.29.2

package/lib/replicator.js

@@ -1059,12 +1059,22 @@ class Peer {
     }
   }
 
+  isActive () {
+    if (this.paused || this.removed) return false
+    return true
+  }
+
   async _send (req) {
     const fork = this.core.tree.fork
 
     this.inflight++
     this.replicator._inflight.add(req)
 
+    if (req.upgrade !== null && req.fork === fork) {
+      const u = this.replicator._addUpgrade()
+      u.inflight.push(req)
+    }
+
     try {
       if (req.block !== null && req.fork === fork) {
         req.block.nodes = await this.core.tree.missingNodes(2 * req.block.index)
@@ -1084,11 +1094,6 @@ class Peer {
       return
     }
 
-    if (req.upgrade !== null && req.fork === fork) {
-      const u = this.replicator._addUpgrade()
-      u.inflight.push(req)
-    }
-
     this.wireRequest.send(req)
   }
 }
@@ -1414,6 +1419,7 @@ module.exports = class Replicator {
 
   _removePeer (peer) {
     this.peers.splice(this.peers.indexOf(peer), 1)
+    peer.removed = true
 
     if (this._manifestPeer === peer) this._manifestPeer = null
 
@@ -1765,7 +1771,7 @@ module.exports = class Replicator {
   }
 
   _updatePeer (peer) {
-    if (peer.paused || peer.inflight >= peer.getMaxInflight()) {
+    if (!peer.isActive() || peer.inflight >= peer.getMaxInflight()) {
       return false
     }
 
@@ -1797,7 +1803,7 @@ module.exports = class Replicator {
   }
 
   _updatePeerNonPrimary (peer) {
-    if (peer.paused || peer.inflight >= peer.getMaxInflight()) {
+    if (!peer.isActive() || peer.inflight >= peer.getMaxInflight()) {
       return false
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hypercore",
-  "version": "10.29.1",
+  "version": "10.29.2",
   "description": "Hypercore is a secure, distributed append-only log",
   "main": "index.js",
   "scripts": {