Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(share/discovery): discovery should try to connect to empty hosts peers #2575

Merged
merged 2 commits into from
Aug 18, 2023
Merged

fix(share/discovery): discovery should try to connect to empty hosts peers #2575

merged 2 commits into from
Aug 18, 2023

Conversation

walldiss
Copy link
Member

libp2p routed host will attempt to find address, if none are provided. First in Peerstore, then by calling FindPeer:

// if we were given some addresses, keep + use them.
	if len(pi.Addrs) > 0 {
		rh.Peerstore().AddAddrs(pi.ID, pi.Addrs, peerstore.TempAddrTTL)
	}

	// Check if we have some addresses in our recent memory.
	addrs := rh.Peerstore().Addrs(pi.ID)
	if len(addrs) < 1 {
		// no addrs? find some with the routing system.
		var err error
		addrs, err = rh.findPeerAddrs(ctx, pi.ID)
		if err != nil {
			return err
		}
	}

It is worth to try to connect in such case.

@walldiss walldiss added area:p2p kind:fix Attached to bug-fixing PRs labels Aug 17, 2023
@walldiss walldiss self-assigned this Aug 17, 2023
@walldiss walldiss merged commit c2c0827 into celestiaorg:main Aug 18, 2023
12 of 14 checks passed
renaynay pushed a commit to renaynay/celestia-node that referenced this pull request Aug 23, 2023
@walldiss @renaynay
fix(share/discovery): discovery should try to connect to empty hosts …
66cd5ff 
…peers (celestiaorg#2575)

libp2p routed host will attempt to find address, if none are provided.
First in `Peerstore`, then by calling `FindPeer`:
```
// if we were given some addresses, keep + use them.
	if len(pi.Addrs) > 0 {
		rh.Peerstore().AddAddrs(pi.ID, pi.Addrs, peerstore.TempAddrTTL)
	}

	// Check if we have some addresses in our recent memory.
	addrs := rh.Peerstore().Addrs(pi.ID)
	if len(addrs) < 1 {
		// no addrs? find some with the routing system.
		var err error
		addrs, err = rh.findPeerAddrs(ctx, pi.ID)
		if err != nil {
			return err
		}
	}
```
It is worth to try to connect in such case.
walldiss added a commit to walldiss/celestia-node that referenced this pull request Sep 22, 2023
@walldiss
fix(share/discovery): discovery should try to connect to empty hosts …
e973b4a 
…peers (celestiaorg#2575)

libp2p routed host will attempt to find address, if none are provided.
First in `Peerstore`, then by calling `FindPeer`:
```
// if we were given some addresses, keep + use them.
	if len(pi.Addrs) > 0 {
		rh.Peerstore().AddAddrs(pi.ID, pi.Addrs, peerstore.TempAddrTTL)
	}

	// Check if we have some addresses in our recent memory.
	addrs := rh.Peerstore().Addrs(pi.ID)
	if len(addrs) < 1 {
		// no addrs? find some with the routing system.
		var err error
		addrs, err = rh.findPeerAddrs(ctx, pi.ID)
		if err != nil {
			return err
		}
	}
```
It is worth to try to connect in such case.

(cherry picked from commit c2c0827)
walldiss added a commit that referenced this pull request Sep 22, 2023
@walldiss
fix(share/discovery): discovery should try to connect to empty hosts …
f4ca00c 
…peers (#2575)

libp2p routed host will attempt to find address, if none are provided.
First in `Peerstore`, then by calling `FindPeer`:
```
// if we were given some addresses, keep + use them.
	if len(pi.Addrs) > 0 {
		rh.Peerstore().AddAddrs(pi.ID, pi.Addrs, peerstore.TempAddrTTL)
	}

	// Check if we have some addresses in our recent memory.
	addrs := rh.Peerstore().Addrs(pi.ID)
	if len(addrs) < 1 {
		// no addrs? find some with the routing system.
		var err error
		addrs, err = rh.findPeerAddrs(ctx, pi.ID)
		if err != nil {
			return err
		}
	}
```
It is worth to try to connect in such case.

(cherry picked from commit c2c0827)
walldiss added a commit to walldiss/celestia-node that referenced this pull request Sep 22, 2023
@walldiss
fix(share/discovery): discovery should try to connect to empty hosts …
7930a13 
…peers (celestiaorg#2575)

libp2p routed host will attempt to find address, if none are provided.
First in `Peerstore`, then by calling `FindPeer`:
```
// if we were given some addresses, keep + use them.
	if len(pi.Addrs) > 0 {
		rh.Peerstore().AddAddrs(pi.ID, pi.Addrs, peerstore.TempAddrTTL)
	}

	// Check if we have some addresses in our recent memory.
	addrs := rh.Peerstore().Addrs(pi.ID)
	if len(addrs) < 1 {
		// no addrs? find some with the routing system.
		var err error
		addrs, err = rh.findPeerAddrs(ctx, pi.ID)
		if err != nil {
			return err
		}
	}
```
It is worth to try to connect in such case.

(cherry picked from commit c2c0827)
walldiss added a commit to walldiss/celestia-node that referenced this pull request Sep 25, 2023
@walldiss
fix(share/discovery): discovery should try to connect to empty hosts …
d6f473c 
…peers (celestiaorg#2575)

libp2p routed host will attempt to find address, if none are provided.
First in `Peerstore`, then by calling `FindPeer`:
```
// if we were given some addresses, keep + use them.
	if len(pi.Addrs) > 0 {
		rh.Peerstore().AddAddrs(pi.ID, pi.Addrs, peerstore.TempAddrTTL)
	}

	// Check if we have some addresses in our recent memory.
	addrs := rh.Peerstore().Addrs(pi.ID)
	if len(addrs) < 1 {
		// no addrs? find some with the routing system.
		var err error
		addrs, err = rh.findPeerAddrs(ctx, pi.ID)
		if err != nil {
			return err
		}
	}
```
It is worth to try to connect in such case.

(cherry picked from commit c2c0827)
@faddat faddat mentioned this pull request Feb 22, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Wondertan Wondertan Wondertan approved these changes

@distractedm1nd distractedm1nd distractedm1nd approved these changes

@renaynay renaynay Awaiting requested review from renaynay renaynay is a code owner

@vgonkivs vgonkivs Awaiting requested review from vgonkivs vgonkivs is a code owner

Assignees

@walldiss walldiss

Labels
area:p2p kind:fix Attached to bug-fixing PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@walldiss @distractedm1nd @Wondertan