Skip to content

fix: Shut down NetworkManager when reloading the domain #4068

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.

Sign up for GitHub

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

Open
simon-lemay-unity wants to merge 2 commits into
base: develop-2.0.0
Choose a base branch
from
+22 −0
Open

fix: Shut down NetworkManager when reloading the domain #4068

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
5fae671
fix: Shutdown NetworkManager when reloading the domain
simon-lemay-unity Jul 6, 2026
ee90d32
Add PR number to CHANGELOG entry
simon-lemay-unity Jul 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions com.unity.netcode.gameobjects/CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ Additional documentation and release notes are available at [Multiplayer Documen

### Fixed

- `NetworkManager` will now perform a proper shutdown when the domain is reloaded in play mode, instead of being left in some half-initialized state that leaks memory and socket handles. (#4068)
- Issue when FastBufferReader is attempting to read a string and all or a portion of the character count has already been read by user script, it could read a character length that results in a negative byte length which could result in an editor crash. (#4052)
- Issue where NetworkRigidbodyBase was not applying rotation correctly when using Rigidbody2D. (#4012)
- Issue where NetworkRigidbodyBase was always checking the 3D rigid body's interpolation mode when determining if it is kinematic and needs to put the rigid body to sleep and then switch to interpolation. (#4012)
Expand Down
21 changes: 21 additions & 0 deletions com.unity.netcode.gameobjects/Editor/NetworkManagerHelper.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,10 @@ private static void InitializeOnload()
Singleton = new NetworkManagerHelper();

NetworkManager.NetworkManagerHelper = Singleton;

AssemblyReloadEvents.beforeAssemblyReload -= AssemblyReloadEvents_beforeAssemblyReload;
AssemblyReloadEvents.beforeAssemblyReload += AssemblyReloadEvents_beforeAssemblyReload;

EditorApplication.playModeStateChanged -= EditorApplication_playModeStateChanged;
EditorApplication.hierarchyChanged -= EditorApplication_hierarchyChanged;

Expand All @@ -55,6 +59,23 @@ private static void InitializeOnload()
};
}

private static void AssemblyReloadEvents_beforeAssemblyReload()
{
if (Application.isPlaying)
{
#if UNITY_2023_1_OR_NEWER

@michalChrobot michalChrobot Jul 7, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

NGOv2.X offers support from 6000.0 editor so I don't think this if is needed

var networkManager = Object.FindAnyObjectByType<NetworkManager>();
#else
var networkManager = Object.FindObjectOfType<NetworkManager>();
#endif
if (networkManager != null && (networkManager.IsServer || networkManager.IsClient))
{
networkManager.Shutdown();
networkManager.ShutdownInternal();
}
}
}

private static void EditorApplication_playModeStateChanged(PlayModeStateChange playModeStateChange)
{
switch (playModeStateChange)
Expand Down