Mirror网络框架中ServerChangeScene对未认证连接的处理问题分析

问题背景

在Mirror网络框架中，ServerChangeScene()方法在服务器切换场景时会调用NetworkServer.SetAllClientsNotReady()，该方法会向所有客户端发送NotReadyMessage。然而，这一机制存在一个潜在问题：它会向尚未完成认证过程的客户端连接也发送NotReadyMessage，导致这些连接被意外断开。

问题现象

当服务器调用ServerChangeScene()方法时，会触发以下行为：

1. 遍历所有客户端连接
2. 对每个连接调用SetClientNotReady()
3. 发送NotReadyMessage给每个客户端

问题在于，即使某些客户端尚未完成认证过程（如正在使用认证器进行认证），也会收到NotReadyMessage。由于NotReadyMessage是专为已认证客户端设计的消息类型，框架会检测到未认证客户端收到了不该接收的消息，从而主动断开这些连接。

技术细节分析

在Mirror框架的NetworkServer类中，SetAllClientsNotReady()方法的实现如下：

foreach (NetworkConnectionToClient conn in connections.Values)
{
    SetClientNotReady(conn);
}

这种方法会无差别地处理所有连接，包括那些正在进行认证但尚未完成的连接。当这些未认证连接收到NotReadyMessage时，框架会记录如下警告并断开连接：

Disconnecting connection: connection(0). Received message Mirror.NotReadyMessage that required authentication, but the user has not authenticated yet

典型场景重现

这个问题在以下特定场景中尤为明显：

1. 使用带有认证器的网络管理器
2. 网络管理器未设置在线场景(onlineScene)
3. 调用NetworkManager.StartHost()启动主机
4. 立即调用ServerChangeScene()切换场景

在这种情况下，主机玩家会被添加到连接列表但尚未完成认证，随后收到NotReadyMessage导致断开连接。

解决方案

针对此问题，有两种可能的修复方案：

方案一：在SetAllClientsNotReady()中跳过未认证连接

public static void SetAllClientsNotReady()
{
    foreach (NetworkConnectionToClient conn in connections.Values)
    {
        if (!conn.isAuthenticated)
            continue;

        SetClientNotReady(conn);
    }
}

方案二：在SetClientNotReady()中增加认证检查

public static void SetClientNotReady(NetworkConnectionToClient conn)
{
    if (!conn.isAuthenticated)
        return;

    conn.isReady = false;
    conn.RemoveFromObservingsObservers();
    conn.Send(new NotReadyMessage());
}

框架更新情况

值得注意的是，这个问题已经在Mirror 89.0.0版本中得到修复。新版本中已经加入了适当的认证检查，确保不会向未认证连接发送NotReadyMessage。开发者如果遇到此问题，应考虑升级到最新版本的Mirror框架。

总结

这个问题揭示了在网络框架设计中需要考虑连接状态的重要性。特别是在处理认证流程时，必须仔细区分已认证和未认证连接，避免向未认证连接发送仅限已认证连接使用的消息。这种设计考量对于构建稳定可靠的网络应用至关重要。