Skip to content

fix: Fixed an exception when EnsureNetworkVariableLengthSafety is enabled #1487

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

Conversation

ShadauxCat
Copy link
Collaborator

MTT-1845
fixes #1428

Changelog

com.unity.netcode.gameobjects

  • Fixed: Fixed an exception being thrown during NetworkVariableDeltaMessage serialization when EnsureNetworkVariableLengthSafety is enabled

Testing and Documentation

  • Includes integration tests.
  • No documentation changes or additions were necessary.

@ShadauxCat ShadauxCat changed the title Fix: Fixed an exception when EnsureNetworkVariableLengthSafety is enabled fix: Fixed an exception when EnsureNetworkVariableLengthSafety is enabled Dec 1, 2021
@ShadauxCat ShadauxCat enabled auto-merge (squash) December 7, 2021 20:56
ShadauxCat and others added 4 commits December 7, 2021 14:57
…rkVariableDeltaMessage.cs

Co-authored-by: Fatih Mar <mfatihmar@gmail.com>
…ety_enabled

# Conflicts:
#	com.unity.netcode.gameobjects/CHANGELOG.md
…ength_safety_enabled' into fix/error_when_networkvariable_length_safety_enabled
@ShadauxCat ShadauxCat merged commit 59e8874 into develop Dec 7, 2021
@ShadauxCat ShadauxCat deleted the fix/error_when_networkvariable_length_safety_enabled branch December 7, 2021 21:30
@ShadauxCat
Copy link
Collaborator Author

Sorry to those who just got added to this already closed review by my cat walking on my keyboard.

@0xFA11
Copy link
Contributor

0xFA11 commented Feb 17, 2022

apparently, #1653 got this backported to develop already.

mollstam pushed a commit to Keepsake-Games/com.unity.netcode.gameobjects that referenced this pull request Feb 13, 2023
…bled (Unity-Technologies#1487)

* Fix: Fixed an exception when EnsureNetworkVariableLengthSafety is enabled

* changelog
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.

Iam Getting error when i am trying to show Player's count
2 participants