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.

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

feat(kad): Return multiaddrs alognside peer IDs #5475

Merged
merged 4 commits into from
Jun 21, 2024

Conversation

Wiezzel
Copy link
Contributor

@Wiezzel Wiezzel commented Jun 19, 2024

Description

Notes & open questions

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • A changelog entry has been made in the appropriate crates

@Wiezzel Wiezzel force-pushed the wiezzel/kad_addrs branch from 1027356 to 2723ca2 Compare June 19, 2024 19:04
@Wiezzel
Copy link
Contributor Author

Wiezzel commented Jun 19, 2024

@guillaumemichel

@Wiezzel Wiezzel force-pushed the wiezzel/kad_addrs branch from 2723ca2 to 92b5b92 Compare June 19, 2024 19:06
@guillaumemichel guillaumemichel self-requested a review June 20, 2024 09:02
protocols/kad/src/behaviour.rs Outdated Show resolved Hide resolved
protocols/kad/src/behaviour.rs Outdated Show resolved Hide resolved
Wiezzel and others added 2 commits June 20, 2024 15:37
Co-authored-by: Guillaume Michel <guillaumemichel@users.noreply.github.com>
@Wiezzel Wiezzel requested a review from guillaumemichel June 21, 2024 07:22
@Wiezzel Wiezzel force-pushed the wiezzel/kad_addrs branch 2 times, most recently from b6e277a to 23592ae Compare June 21, 2024 07:31
@Wiezzel Wiezzel force-pushed the wiezzel/kad_addrs branch from 23592ae to 8d6fb8b Compare June 21, 2024 07:32
Copy link
Contributor

@guillaumemichel guillaumemichel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you @Wiezzel!

@mergify mergify bot merged commit e28de77 into libp2p:master Jun 21, 2024
71 of 72 checks passed
@stormshield-frb
Copy link
Contributor

stormshield-frb commented Jun 21, 2024

Sorry to review after merge but why increase the version of libp2p-kad to 0.47.0 ? The 0.46.0 is not released yet if I am not mistaken. Same thing for the libp2p-core.

@Wiezzel Wiezzel deleted the wiezzel/kad_addrs branch June 21, 2024 12:19
@Wiezzel
Copy link
Contributor Author

Wiezzel commented Jun 21, 2024

@stormshield-frb I'm sorry. I'm no up-to-date with the release schedule.

@guillaumemichel
Copy link
Contributor

Sorry to review after merge but why increase the version of libp2p-kad to 0.47.0 ? The 0.46.0 is not released yet if I am not mistaken. Same thing for the libp2p-core.

Good point! We'll address it in a follow up PR shortly.

guillaumemichel added a commit that referenced this pull request Jun 25, 2024
@guillaumemichel guillaumemichel mentioned this pull request Jun 25, 2024
4 tasks
mergify bot pushed a commit that referenced this pull request Jun 25, 2024
Correct `kad` changelog after #5475

`kad` `0.46.0` isn't released yet, hence changelog entry from #5475 will be published with `0.46.0`.

Pull-Request: #5478.
TimTinkers pushed a commit to unattended-backpack/rust-libp2p that referenced this pull request Sep 14, 2024
TimTinkers pushed a commit to unattended-backpack/rust-libp2p that referenced this pull request Sep 14, 2024
Correct `kad` changelog after libp2p#5475

`kad` `0.46.0` isn't released yet, hence changelog entry from libp2p#5475 will be published with `0.46.0`.

Pull-Request: libp2p#5478.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants