Skip to content

fix: support handshake-only with no muxers #519

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

Merged
merged 1 commit into from
Apr 23, 2025

Conversation

achingbrain
Copy link
Collaborator

Allows transports such as WebRTC-Direct to skip the early muxer negotiation.

This handles an edge case where you have a libp2p node configured with no dedicated muxer(s), instead transport(s) that incorporate their own muxers but that use Noise to exchange PeerIDs.

Splits the existing muxer negotiation test out into it's own file and adds more tests to handle the various configurations.

@achingbrain achingbrain requested a review from a team as a code owner April 16, 2025 12:17
Allows transports such as WebRTC-Direct to skip the early muxer
negotiation.

This handles an edge case where you have a libp2p node
configured with no dedicated muxer(s), instead transport(s) that
incorporate their own muxers but that use Noise to exchange PeerIDs.

Splits the existing muxer negotiation test out into it's own file
and adds more tests to handle the various configurations.
@achingbrain achingbrain force-pushed the fix/support-handshake-only-with-no-muxers branch from 6e14ab4 to 05ee3bd Compare April 16, 2025 12:17
@achingbrain achingbrain merged commit fd0e4f2 into master Apr 23, 2025
18 checks passed
@achingbrain achingbrain deleted the fix/support-handshake-only-with-no-muxers branch April 23, 2025 14:32
wemeetagain pushed a commit that referenced this pull request Apr 23, 2025
## [16.1.3](v16.1.2...v16.1.3) (2025-04-23)

### Bug Fixes

* support handshake-only with no muxers ([#519](#519)) ([fd0e4f2](fd0e4f2))
@wemeetagain
Copy link
Member

🎉 This PR is included in version 16.1.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

2 participants