hldb / welo

peer-to-peer, collaborative states using Merkle-CRDTs
Other
32 stars 2 forks source link

deps: Update libp2p, helia and their modules. #70

Closed saul-jb closed 1 year ago

saul-jb commented 1 year ago

Libp2p had some breaking changes in the update from 0.44 to 0.45, which has resulted in Helia and GossipSub needing updates. This PR updates these packages and types to the new versions.

Creating as a draft because this update seems to cause timeout issues in the test-direct.ts test due to Direct not triggering events.

tabcat commented 1 year ago

This is seriously great :+1: have been anticipating libp2p 0.45 upgrade

will checkout branch and see if I can fix issue with pubsub

tabcat commented 1 year ago

For some reason the libp2p nodes are not able to connect

tabcat commented 1 year ago

Ah I think I need to add some services to the Libp2p config (e.g. identify service)

tabcat commented 1 year ago

Everything passing locally although less reliably, the direct test can timeout in the first before hook and I've seen live-replicator timeout in replicates replica entries and identities test

tabcat commented 1 year ago

Going to merge for now. Might need to fix the timeouts in the direct and live-replicator tests if they arent reliable. It seems like there might be a race condition being tripped when 2 nodes dial each other at the same time which hangs both of them. (not sure though)