Skip to content

fix: cleaning up connected clients upon shutdown [MTT-1573] (backport #1945) #1949

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 5 commits into from
May 9, 2022

Conversation

jeffreyrainy
Copy link
Contributor

Backport of:
This cleans up the connected clients upon shutdown. Proper clean-up.

Doesn't change much for the customers, though, since recent Netcode for GameObjects would throw an exception, anyway, if trying to access connected clients while not hosting.

#416

@jeffreyrainy jeffreyrainy requested a review from a team as a code owner May 9, 2022 20:39
@jeffreyrainy jeffreyrainy requested review from 0xFA11 and ashwinimurt May 9, 2022 20:39
@@ -10,6 +10,9 @@ Additional documentation and release notes are available at [Multiplayer Documen

- Fixed: Hosting again after failing to host now works correctly

- Fixed NetworkManager to cleanup connected client lists after stopping (#1945)
- Fixed: NetworkHide followed by NetworkShow on the same frame works correctly (#1940)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huh?

@0xFA11 0xFA11 changed the title fix: cleaning up connected clients upon shutdown [MTT-1573] #1945 backport fix: cleaning up connected clients upon shutdown [MTT-1573] (backport #1945) May 9, 2022
@0xFA11 0xFA11 enabled auto-merge (squash) May 9, 2022 21:40
@0xFA11 0xFA11 merged commit ca5b3a8 into release/1.0.0 May 9, 2022
@0xFA11 0xFA11 deleted the chore/client-cleanup-backport branch May 9, 2022 22:06
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