Skip to content

net/litep2p: Bring the latest compatibility fixes to litep2p#7405

Closed
lexnv wants to merge 10 commits intomasterfrom
lexnv/update-litep2p-compat
Closed

net/litep2p: Bring the latest compatibility fixes to litep2p#7405
lexnv wants to merge 10 commits intomasterfrom
lexnv/update-litep2p-compat

Conversation

@lexnv
Copy link
Copy Markdown
Contributor

@lexnv lexnv commented Jan 31, 2025

This PR updates litep2p to a git version for testing the compatibility between libp2p <-> litep2p.

Litep2p uses the new implementation of yamux which contains many fixes and performance improvements.
At the same time, a few fixes for multistream select compatibilities might have triggered libp2p to not consider the request timeouts.

Changes

  • yamux: Switch to upstream implementation while keeping the controller API (#320)
  • multistream-select: Enforce io::error instead of empty protocols (#318)
  • multistream: Do not wait for negotiation in poll_close (#319)

lexnv added 2 commits January 31, 2025 11:26
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@paritytech-workflow-stopper
Copy link
Copy Markdown

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/13161688865
Failed job name: check-runtime-migration

lexnv and others added 7 commits February 5, 2025 18:27
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
…p-compat

Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@lexnv
Copy link
Copy Markdown
Contributor Author

lexnv commented Feb 20, 2025

Closed by #7640

@lexnv lexnv closed this Feb 20, 2025
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.

1 participant