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

refactor: update libp2p-yamux and fix breaking changes #121

Merged
merged 1 commit into from
Feb 10, 2024
Merged

Conversation

paralin
Copy link
Member

@paralin paralin commented Dec 1, 2023

refactor: update libp2p-yamux and fix breaking changes

  • Passing a logger to MuxerInit is now required
  • Passing a logger to yamux()({logger:...}) is now required

See: #120

Update dependencies and fix various breaking changes introduced by those
dependency updates:

  • Change type of Conn to Duplex<Uint8Array | Uint8ArrayList>
  • Pass a logger stub to yamux as it's now required.

Related:

@paralin paralin marked this pull request as ready for review January 10, 2024 21:05
@paralin paralin marked this pull request as draft January 10, 2024 21:09
@paralin paralin force-pushed the update-yamux branch 2 times, most recently from cd11abb to dbe29fa Compare January 17, 2024 22:30
@paralin paralin force-pushed the update-yamux branch 4 times, most recently from 7b8b19d to fa22db9 Compare February 10, 2024 01:24
@paralin paralin marked this pull request as ready for review February 10, 2024 01:25
@paralin paralin changed the title [WIP] refactor: update libp2p-yamux and fix breaking changes refactor: update libp2p-yamux and fix breaking changes Feb 10, 2024
@paralin paralin force-pushed the update-yamux branch 3 times, most recently from e02f163 to aa52059 Compare February 10, 2024 01:31
 - Passing a logger to MuxerInit is now required
 - Passing a logger to yamux()({logger:...}) is now required

See: #120

Update dependencies and fix various breaking changes introduced by those
dependency updates:

- Change type of Conn to Duplex<Uint8Array | Uint8ArrayList>
- Pass default logger to yamux as it's now required.
 - We need to pass a Uint8ArrayList to the WebSocket it-ws.

Related:

- ChainSafe/js-libp2p-yamux#69
- libp2p/js-libp2p#2275
- ChainSafe/js-libp2p-yamux#70

Signed-off-by: Christian Stewart <[email protected]>
@paralin paralin merged commit a2e6917 into main Feb 10, 2024
4 checks passed
@paralin paralin deleted the update-yamux branch February 10, 2024 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant