Skip to content

Update libp2p to v0.22.0#503

Merged
tzdybal merged 4 commits intomainfrom
tzdybal/libp2p
Sep 6, 2022
Merged

Update libp2p to v0.22.0#503
tzdybal merged 4 commits intomainfrom
tzdybal/libp2p

Conversation

@tzdybal
Copy link
Copy Markdown
Contributor

@tzdybal tzdybal commented Aug 31, 2022

This PR updates go-libp2p, go-libp2p-pubsub and go-libp2p-kad-dht to the latest versions.

Resolves #502.

@tzdybal tzdybal requested review from Wondertan and jbowen93 August 31, 2022 19:34
@tzdybal tzdybal self-assigned this Aug 31, 2022
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #503 (0c62d0c) into main (539f460) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #503   +/-   ##
=======================================
  Coverage   56.09%   56.09%           
=======================================
  Files          50       50           
  Lines        9279     9279           
=======================================
  Hits         5205     5205           
  Misses       3314     3314           
  Partials      760      760           
Impacted Files Coverage Δ
block/manager.go 62.35% <ø> (ø)
conv/crypto.go 76.92% <ø> (ø)
node/mempool.go 50.00% <ø> (ø)
node/node.go 61.11% <ø> (ø)
p2p/client.go 62.91% <ø> (ø)
p2p/gossip.go 82.00% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Comment thread .golangci.yml Outdated
revive:
rules:
- name: package-comments
disabled: true No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you clean up this last line? Weird that the red thing is on a different line. Perhaps a Windows/Unix newline thing?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changes pushed, and IDE settings corrected. 👍

@tzdybal tzdybal enabled auto-merge September 6, 2022 09:28
Copy link
Copy Markdown
Contributor

@Wondertan Wondertan left a comment

Choose a reason for hiding this comment

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

Sweet, I saw how they were cleaning up dependencies, but now that's visible on the go.sum. libp2p becomes lightweight now

Changes themselves look good, and I don't see anything wrong. The libp2p update is mostly about moving stuff back and forth with some chores.

Another recently changed is the default identity keys, which are now ed25565 by default rather than RSA. If you explicitly use ed25562 in the libp2p.Host construction, then everything is fine

@tzdybal tzdybal merged commit 41d5cc3 into main Sep 6, 2022
@tzdybal tzdybal deleted the tzdybal/libp2p branch September 6, 2022 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update to libp2p v0.22.0

4 participants