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

Go back to upstream libp2p-websocket-websys #2803

Merged
merged 1 commit into from
Aug 5, 2024
Merged

Conversation

sisou
Copy link
Member

@sisou sisou commented Aug 5, 2024

As they fixed the issue in v0.3.3: libp2p/rust-libp2p#5521 (comment)

This fixes libp2p/rust-libp2p#5490.

Pull request checklist

  • All tests pass. The project builds and runs.
  • I have resolved any merge conflicts.
  • I have resolved all clippy and rustfmt warnings.

@sisou sisou self-assigned this Aug 5, 2024
@nibhar
Copy link
Member

nibhar commented Aug 5, 2024

What about #2301? Is this going to be closed by the upstream as well?

@sisou
Copy link
Member Author

sisou commented Aug 5, 2024

@nibhar No. Unfortunately, in libp2p/rust-libp2p#4889 they made us change the implementation from being runtime agnostic to only accepting environments with a Window or WorkerGlobalScope (scopes supported by the web-sys crate). And Node doesn't have those. So that workaround is kept.

@hrxi hrxi merged commit d037621 into albatross Aug 5, 2024
7 checks passed
@hrxi hrxi deleted the sisou/websocket-websys branch August 5, 2024 12:26
@hrxi
Copy link
Member

hrxi commented Aug 5, 2024

Thanks!

@jsdanielh jsdanielh added this to the Nimiq PoS Mainnet milestone Aug 7, 2024
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.

websocket-websys: Use-after-free of onclose and onerror event handlers
5 participants