NetworkedObject Ownership Management

Description: The ownership lifecycle seems to have a gap in it. We have an object whose ownership we would like to pass from player to player. The ownership transfer is initiated by each individual player when specific conditions are met.

MLAPI prevents non-server clients from requesting ownership transfer. This throws an error when called on clients:

   public void ChangeOwner(ulong ownerID) {
      this.NetworkedObject.ChangeOwnership(ownerID);
    }

NotServerException: Only the server can change ownership
MLAPI.Spawning.SpawnManager.ChangeOwnership (MLAPI.NetworkedObject netObject, System.UInt64 clientId) (at C:/projects/mlapi/MLAPI/Spawning/SpawnManager.cs:207)

Additionally, setting up a server-side ownership transfer method also throws error (because I’m guessing the ulong ownerID argument is being confused with the ulong method hash and wires are getting crossed):

    public void ChangeOwner(ulong ownerID) {
      InvokeServerRpc(this.ServerRPCOwnerChange, ownerID);
    }

    [ServerRPC]
    public void ServerRPCOwnerChange(ulong ownerID) {
      this.NetworkedObject.ChangeOwnership(ownerID);
    }

NullReferenceException: Object reference not set to an instance of an object
MLAPI.NetworkedBehaviour.InvokeServerRPCLocal (System.UInt64 hash, System.UInt64 senderClientId, System.IO.Stream stream) (at C:/projects/mlapi/MLAPI/Core/NetworkedBehaviour.cs:708)
MLAPI.NetworkedBehaviour.SendServerRPCPerformance (System.UInt64 hash, System.IO.Stream messageStream, System.String channel, MLAPI.Security.SecuritySendFlags security) (at C:/projects/mlapi/MLAPI/Core/NetworkedBehaviour.cs:843)
MLAPI.NetworkedBehaviour.SendServerRPCBoxed (System.UInt64 hash, System.String channel, MLAPI.Security.SecuritySendFlags security, System.Object[] parameters) (at C:/projects/mlapi/MLAPI/Core/NetworkedBehaviour.cs:739)
MLAPI.NetworkedBehaviour.InvokeServerRpc[T1] (MLAPI.NetworkedBehaviour+RpcMethod`1[T1] method, T1 t1, System.String channel, MLAPI.Security.SecuritySendFlags security) (at C:/projects/mlapi/MLAPI/Core/RPCMethods/NetworkedBehaviour.BoxedServerRpc.cs:32)

Expected Behavior: Hoping that it’s within the scope of your intended ownership control that we can pass ownership between players and player/server somewhat willy nilly.

Environment:

• OS: Windows 10
• Unity Version: 2018.4.9f1
• MLAPI Version: 11.2.0

[slumtrimpet]

Yes, thank you for your support via Discord. I added a bunch more logging and found that I was adding a NetworkedObject at runtime and the order in which I was doing that was screwing up everything. The problem was 100% on my side and MLAPI is amazing. 😃 Thanks!

[TwoTenPvP]

Has this been solved?