hypercore 11.13.1 → 11.13.3

package/index.js

@@ -1112,7 +1112,7 @@ function initOnce (session, storage, key, opts) {
 
   session.core = new Core(Hypercore.defaultStorage(storage), {
     preopen: opts.preopen,
-    eagerUpgrade: true,
+    eagerUpgrade: opts.eagerUpgrade !== false,
     notDownloadingLinger: opts.notDownloadingLinger,
     allowFork: opts.allowFork !== false,
     inflightRange: opts.inflightRange,

package/lib/replicator.js

@@ -30,7 +30,7 @@ const ReceiverQueue = require('./receiver-queue')
 const HotswapQueue = require('./hotswap-queue')
 const RemoteBitfield = require('./remote-bitfield')
 const { MerkleTree } = require('./merkle-tree')
-const { REQUEST_CANCELLED, REQUEST_TIMEOUT, INVALID_CAPABILITY, SNAPSHOT_NOT_AVAILABLE } = require('hypercore-errors')
+const { REQUEST_CANCELLED, REQUEST_TIMEOUT, INVALID_CAPABILITY, SNAPSHOT_NOT_AVAILABLE, ASSERTION } = require('hypercore-errors')
 const m = require('./messages')
 const caps = require('./caps')
 
@@ -429,6 +429,8 @@ class Peer {
 
     this.needsSync = false
     this.syncsProcessing = 0
+    this.lastUpgradableLength = 0
+    this.lastUpgradableFork = 0
 
     this._remoteContiguousLength = 0
 
@@ -662,6 +664,10 @@ class Peer {
 
     if (length === this.remoteLength && fork === this.core.state.fork) {
       this.canUpgrade = upgrade
+      if (upgrade) {
+        this.lastUpgradableFork = fork
+        this.lastUpgradableLength = length
+      }
     }
 
     if (--this.syncsProcessing !== 0) return // ie not latest
@@ -683,16 +689,26 @@ class Peer {
   async _updateCanUpgradeAndSync () {
     const { length, fork } = this.core.state
 
+    const remoteLength = this.remoteLength
+    const remoteFork = this.remoteFork
     const canUpgrade = await this._canUpgrade(this.remoteLength, this.remoteFork)
 
     if (this.syncsProcessing > 0 || length !== this.core.state.length || fork !== this.core.state.fork) {
       return
     }
+    if (remoteLength !== this.remoteLength || remoteFork !== this.remoteFork) {
+      return
+    }
     if (canUpgrade === this.canUpgrade) {
       return
     }
 
     this.canUpgrade = canUpgrade
+    if (canUpgrade) {
+      this.lastUpgradableLength = this.remoteLength
+      this.lastUpgradableFork = this.remoteFork
+    }
+
     this.sendSync()
   }
 
@@ -1204,7 +1220,7 @@ class Peer {
 
       if (this._remoteHasBlock(index) === false) continue
       if (this.core.bitfield.get(index) === true) continue
-      if (!this._hasTreeParent(index)) continue
+      if (!this._canRequest(index)) continue
 
       // Check if this block is currently inflight - if so pick another
       const b = this.replicator._blocks.get(index)
@@ -1233,35 +1249,11 @@ class Peer {
     return false
   }
 
-  _hasTreeParent (index) {
+  _canRequest (index) {
+    if (!(index >= 0)) throw ASSERTION('bad index to _canRequest: ' + index)
     if (this.remoteLength >= this.core.state.length) return true
-
-    const ite = flatTree.iterator(index * 2)
-
-    let span = 2
-    let length = 0
-
-    while (true) {
-      ite.parent()
-
-      const left = (ite.index - ite.factor / 2 + 1) / 2
-      length = left + span
-
-      // if larger than local AND larger than remote - they share the root so its ok
-      if (length > this.core.state.length) {
-        if (length > this.remoteLength) return true
-        break
-      }
-
-      // its less than local but larger than remote so skip it
-      if (length > this.remoteLength) break
-
-      span *= 2
-      const first = this.core.bitfield.findFirst(true, left)
-      if (first > -1 && first < length) return true
-    }
-
-    // TODO: push to async queue and check against our local merkle tree if we actually can request this block
+    if (index < this.lastUpgradableLength && this.lastUpgradableFork === this.core.state.fork) return true
+    if (this.canUpgrade && this.syncsProcessing === 0) return true
     return false
   }
 
@@ -1286,9 +1278,7 @@ class Peer {
       return false
     }
 
-    if (!this._hasTreeParent(b.index)) {
-      return false
-    }
+    if (!this._canRequest(b.index)) return false
 
     const req = this._makeRequest(b.index >= length, b.priority, b.index + 1)
     if (req === null) return false
@@ -1299,7 +1289,7 @@ class Peer {
   }
 
   _requestRangeBlock (index, length) {
-    if (this.core.bitfield.get(index) === true || !this._hasTreeParent(index)) return false
+    if (this.core.bitfield.get(index) === true || !this._canRequest(index)) return false
 
     const b = this.replicator._blocks.add(index, PRIORITY.NORMAL)
     if (b.inflight.length > 0) {

package/package.json

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