Skip to content

fix: NGO UnityTransport not detecting invalid endpoint [MTT-3937]#2496

Merged
NoelStephensUnity merged 7 commits intodevelopfrom
fix/not-detecting-invalid-endpoint-mtt-3937
Apr 10, 2023
Merged

fix: NGO UnityTransport not detecting invalid endpoint [MTT-3937]#2496
NoelStephensUnity merged 7 commits intodevelopfrom
fix/not-detecting-invalid-endpoint-mtt-3937

Conversation

@NoelStephensUnity
Copy link
Copy Markdown
Member

@NoelStephensUnity NoelStephensUnity commented Apr 7, 2023

This resolves the issue where the NGO UnityTransport class was not detecting an invalid endpoint address and would always return true (even with an invalid endpoint address).

MTT-3937
Also pertains to issue #2525

Changelog

  • Fixed: Issue where invalid endpoint addresses were not being detected and returning false from NGO UnityTransport.

Testing and Documentation

  • Includes unit test.
  • No documentation changes or additions were necessary.

detect and return early if trying to bind to an invalid endpoint.
Adding a unit test to validate the fix.
updating changelog entry
@NoelStephensUnity NoelStephensUnity requested review from a team as code owners April 7, 2023 20:55
Adding the PR number to changelog entry
Minor tweaks to NetcodeLogAssert.
Updating test to detect invalid endpoint.
Fixing the editor test expecting a error log that no longer will happen with an invalid endpoint address.
@NoelStephensUnity NoelStephensUnity merged commit 4abf5a1 into develop Apr 10, 2023
@NoelStephensUnity NoelStephensUnity deleted the fix/not-detecting-invalid-endpoint-mtt-3937 branch April 10, 2023 21:48
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