I have tried to add Gossipsub to the echo example her

I think this issue got a little worse since <a class="issue-link js-issue-link" data-e

<a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="/us

Please can you try with libp2p@0.46.17 (just released) - it has the fixes for protocol

feat: Support WebRTC Browser-to-Browser about js-libp2p-gossipsub HOT 7 OPEN

DougAnderson444 commented on August 20, 2024 2

feat: Support WebRTC Browser-to-Browser

from js-libp2p-gossipsub.

Comments (7)

justin0mcateer commented on August 20, 2024

I am investigating this issue. GossipSub ignores any new connections to a peer if it already has a connection to that peer:
https://github.com/ChainSafe/js-libp2p-gossipsub/blob/master/src/index.ts#L745

I am going to take a stab at making the stream management more robust by using the AddressSorter mechanism to determine which connection is more 'preferred'.

One would think that once the Relay connection dies, that GossipSub would attempt to re-connect to the peer via the remaining (WebRTC) connection. Not sure why that doesn't happen I will investigate that further today.

from js-libp2p-gossipsub.

Marcel-G commented on August 20, 2024

I think this issue got a little worse since libp2p/js-libp2p#1890
Gossipsub will now not send over relayed (transient) connections and it does not use the upgraded webrtc connection as mentioned in this issue. Effectively it doesn't send at all

from js-libp2p-gossipsub.

justin0mcateer commented on August 20, 2024

I have managed to get it working, but it wasn't easy. The biggest problem is that GossipSub tries to connect in peer discovery, but it gets aborted because it is a transient connection. The default connection logic is that GossipSub basically never tries again.

One work around to this is to enable 'directPeers' in the GossipSub config, this will cause GossipSub to attempt to connect to the listed peers every heartbeat (1 second by default).

from js-libp2p-gossipsub.

Marcel-G commented on August 20, 2024

@justin0mcateer does that workaround involve knowing the directPeers peer ids and multiaddrs up-front or do you have a way of doing this dynamically?

from js-libp2p-gossipsub.

Marcel-G commented on August 20, 2024

@achingbrain I see that runOnTransientConnection was added to handlers, but is it somehow relevant for topologies also?
It feels a little strange we would get onConnect with a connection that cannot be used with the protocol

js-libp2p-gossipsub/src/index.ts

Lines 589 to 592 in a2981f9

 const topology = { 

 onConnect: this.onPeerConnected.bind(this), 

 onDisconnect: this.onPeerDisconnected.bind(this) 

 }

from js-libp2p-gossipsub.

justin0mcateer commented on August 20, 2024

I agree. I was thinking about whether it would make sense to have the Identify service filter out 'non-transient' protocols on transient connections. To minimize superflous new stream requests. Or maybe have GossipSub use 'dialProtocol' and filter connections based on whether the protocol is transient.

…

________________________________ From: Marcel Gleeson ***@***.***> Sent: Tuesday, August 22, 2023 10:05:25 AM To: ChainSafe/js-libp2p-gossipsub ***@***.***> Cc: justin0mcateer ***@***.***>; Mention ***@***.***> Subject: Re: [ChainSafe/js-libp2p-gossipsub] feat: Support WebRTC Browser-to-Browser (Issue #448) I see that runOnTransientConnection was added to handlers, but is it somehow relevant to topologies also? It feels a little strange we would get onConnect with a connection that cannot be used with the protocol https://github.com/ChainSafe/js-libp2p-gossipsub/blob/a2981f956d1f427fae9deb9c769f0baab0db5d26/src/index.ts#L589-L592 — Reply to this email directly, view it on GitHub<#448 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAT4Q7OMG5VGD2SU7HCWQSTXWTDDLANCNFSM6AAAAAAZ6ZCFOY>. You are receiving this because you were mentioned.Message ID: ***@***.***>

from js-libp2p-gossipsub.

achingbrain commented on August 20, 2024

Please can you try with [email protected] (just released) - it has the fixes for protocol handlers & transient streams incorporated - by default gossipsub will no longer run over a transient stream, only direct connections.

from js-libp2p-gossipsub.

feat: Support WebRTC Browser-to-Browser about js-libp2p-gossipsub HOT 7 OPEN

Comments (7)

Related Issues (20)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	const topology = {
	onConnect: this.onPeerConnected.bind(this),
	onDisconnect: this.onPeerDisconnected.bind(this)
	}