Skip to content

fix: Issues when using UTP 2.0.0-exp.6#2224

Merged
simon-lemay-unity merged 3 commits intodevelopfrom
fix/relay-server-endpoint
Sep 28, 2022
Merged

fix: Issues when using UTP 2.0.0-exp.6#2224
simon-lemay-unity merged 3 commits intodevelopfrom
fix/relay-server-endpoint

Conversation

@simon-lemay-unity
Copy link
Copy Markdown
Contributor

This PR fixes two issues that occurred if UTP 2.0.0-exp.6 was installed:

  • Clients using Relay would fail to connect because in exp.6 we consider connecting to endpoints other than the Relay server to be an error. This was reverted in exp.7 but that's not available yet.
  • Simplified the creation of DataStreamReader in BatchedSendQueue so that it works the same way with UTP 1.X and 2.0. This avoids an issue where the temporary native array didn't have an atomic safety handle, causing an exception when creating the stream reader.

Changelog

N/A (None of the issues should be observable by users on released versions of NGO.)

Testing and Documentation

  • No tests have been added.
  • No documentation changes or additions were necessary.

@simon-lemay-unity simon-lemay-unity requested a review from a team as a code owner September 28, 2022 20:45
@simon-lemay-unity simon-lemay-unity enabled auto-merge (squash) September 28, 2022 20:50
@simon-lemay-unity simon-lemay-unity merged commit bc08e65 into develop Sep 28, 2022
@simon-lemay-unity simon-lemay-unity deleted the fix/relay-server-endpoint branch September 28, 2022 21:24
jakobbbb pushed a commit to GooseGirlGames/com.unity.netcode.gameobjects that referenced this pull request Feb 22, 2023
* fix: Bind to server endpoint when using Relay

* Fix NativeArray conversion

* Simplify creation of DataStreamReader
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.

2 participants