Skip to content

chore: expose websockets #2201

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 11 commits into from
Sep 14, 2022
Merged

chore: expose websockets #2201

merged 11 commits into from
Sep 14, 2022

Conversation

jeffreyrainy
Copy link
Contributor

This PR is another step for https://jira.unity3d.com/browse/MTT-4522

It brings us to a point where

  • Users can use WebSocket connections on platforms where it is supported (including WebGL.)
  • WebSocket-based connection works on Desktop platforms
  • WebSocket-based connection works on WebGL platform
  • Additive-only changes, no breaking changes, are introduced

It doesn't yet cover:

  • WebSocket works with Unity Relay

As this is not ready on the Relay side.

In order to use WebSockets, user must check "Use Web Sockets":

image

@jeffreyrainy jeffreyrainy marked this pull request as ready for review September 13, 2022 21:24
@jeffreyrainy jeffreyrainy requested a review from a team as a code owner September 13, 2022 21:24
@simon-lemay-unity
Copy link
Contributor

Should this be noted in the changelog?

@jeffreyrainy jeffreyrainy requested a review from a team as a code owner September 13, 2022 21:52
@jeffreyrainy jeffreyrainy enabled auto-merge (squash) September 14, 2022 03:00
@jeffreyrainy jeffreyrainy merged commit 1046647 into develop Sep 14, 2022
@jeffreyrainy jeffreyrainy deleted the chore/expose-websockets branch September 14, 2022 04:01
jakobbbb pushed a commit to GooseGirlGames/com.unity.netcode.gameobjects that referenced this pull request Feb 22, 2023
* chore: websockets. Minimalist UI to use WebSockets

* fix: adjusting #ifdef on transport version to allow building non-webgl platforms with UTP 2.0

* fix: addressing build issue where WebGL platform requires different name for UTP symbols

* fix: emitting a warning and switching to WebSockets if uers don't, on WebGL platform

* Addressing PR review comments

* Addressing PR review comments

* Update com.unity.netcode.gameobjects/CHANGELOG.md

Co-authored-by: Fatih Mar <[email protected]>

Co-authored-by: Fatih Mar <[email protected]>
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.

3 participants