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

fix(interop/WASM): use selenium/standalone-chrome:115.0 #4383

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

mxinden
Copy link
Member

@mxinden mxinden commented Aug 23, 2023

Description

Our WASM Webtransport interoperability tests previously used Chrome 112. This Chrome version fails to connect to go-libp2p with quic-go v0.38.0. See libp2p/go-libp2p#2506 for failure. This is due to quic-go v0.38.0 moving to the updated code point for HTTP datagrams. See quic-go/quic-go#3588 for details.

This commit upgrades our interop tests to use Chrome 115.

Notes & open questions

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
  • A changelog entry has been made in the appropriate crates

Our WASM Webtransport interoperability tests previously used Chrome 112. This Chrome version fails
to connect to go-libp2p with quic-go v0.38.0. See libp2p/go-libp2p#2506 for
failure. This is due to quic-go v0.38.0 moving to the updated code point for HTTP datagrams. See
quic-go/quic-go#3588 for details.

This commit upgrades our interop tests to use Chrome 115.
@mxinden mxinden added send-it trivial Marks PRs which are considered trivial and don't need approval from another maintainer. labels Aug 23, 2023
@mxinden
Copy link
Member Author

mxinden commented Aug 23, 2023

@mergify backport v0.52

@mergify
Copy link

mergify bot commented Aug 23, 2023

backport v0.52

❌ No backport have been created

  • Backport to branch v0.52 failed

GitHub error: Branch not found

@mxinden mxinden changed the title fix(interop/Dockerfile.chromium): use selenium/standalone-chrome:115.0 fix(interop/WASM): use selenium/standalone-chrome:115.0 Aug 23, 2023
@mergify mergify bot merged commit 54fa53a into libp2p:master Aug 23, 2023
67 of 69 checks passed
@mxinden
Copy link
Member Author

mxinden commented Aug 23, 2023

backport v0.52

❌ No backport have been created

Makes total sense, given that current master has not yet broken with libp2p v0.52.0.

mxinden added a commit to libp2p/test-plans that referenced this pull request Aug 23, 2023
Bump rust-libp2p v0.52 interop version to include
libp2p/rust-libp2p#4383 which fixes issue in
libp2p/go-libp2p#2506.
mxinden added a commit to libp2p/test-plans that referenced this pull request Aug 24, 2023
Bump rust-libp2p v0.52 interop version to include
libp2p/rust-libp2p#4383 which fixes issue in
libp2p/go-libp2p#2506.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
send-it trivial Marks PRs which are considered trivial and don't need approval from another maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant