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

Revert "chore: upgrade libp2p to 0.52.2 (#2732)" #2759

Merged
merged 6 commits into from
Apr 14, 2023

Conversation

LesnyRumcajs
Copy link
Member

@LesnyRumcajs LesnyRumcajs commented Apr 13, 2023

This reverts commit f61e3de.

Summary of changes

Changes introduced in this pull request:

  • reverting the libp2p upgrade that most likely caused the bootstrap failure

Reference issue to close (if applicable)

Fixes #2755

Other information and links

Libp2p update tracking issue: #2762

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 (if possible),
  • I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

@@ -97,7 +97,7 @@ lazy_static = "1.4"
libipld = { version = "0.14", default-features = false, features = ["dag-cbor", "dag-json", "derive"] }
libipld-core = "0.14"
libipld-macro = "0.14"
libp2p = { version = "0.51.2", default-features = false }
Copy link
Contributor

Choose a reason for hiding this comment

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

It will be great to pin libp2p to "=0.51.1" here, or cargo update will break the build

@LesnyRumcajs LesnyRumcajs enabled auto-merge (squash) April 14, 2023 07:17
@LesnyRumcajs LesnyRumcajs merged commit a02c55c into main Apr 14, 2023
@LesnyRumcajs LesnyRumcajs deleted the revert-libp2p-upgrade branch April 14, 2023 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Forest stops bootstrap on NetworkMessageQueryFailed
3 participants