(Solved)SpawnWithClientAuthority doesn't assign LocalPlayerAuthority

Unity Engine
1

Hello,
I have a problem when i want to spawn the player gameobject trought a custom prefab with localplayerauthority that allow to choose the character, but when the 2nd object spawn, it doesn’t have the authority

Im using that:

    [Command]
    public void CmdSpawn()
    {
        GameObject go = (GameObject)Instantiate(playerselected, SpawnPoint2.transform.position, Quaternion.identity);
        NetworkServer.SpawnWithClientAuthority(go, connectionToClient;
    }

Im sure it doesn’t have localplayer… because this script is destroyed on the player

if (!isLocalPlayer)
        {

            AudioListenerCamera.SetActive(false);
            Camera2.enabled = false;
            Destroy(this);
            return;
        }

Can you help me please?

2

Still stuck, note that both prefabs have network identity with localplayer set on inspector

3

Please?

4

I beg you x.x

5

tijan, can you share a little bit more of your code please. With the example code we don’t know you’re using Destroy(this); on the server, instead of the local client.

6

My script got more than 1500 line, destroy(this) destroy the script of player who aren’t suppose to be controlled on client side, it works well at this moment

Here the codes:

using System.Collections;
using System.Collections.Generic;
using UnityEngine.Events;
using UnityEngine;
using UnityEngine.Networking;
using UnityEngine.UI;

public class PlayerController_Net : NetworkBehaviour
{
    public bool canWallJump = true;
    public int currentHealth;
    public Texture viseurTexture;
    public Texture healthTexture;
    public float hpbarlenght;
    public Texture fondTexturebar;
    public Texture ammoBarTexture;
    public float ammobarlenght;
    public float ammobarpercent;
    public Texture manaBarTexture;
    public float manabarlenght;
    public float manabarpercent;
    public GameObject ToPManager;
    public bool GradinCantAttack = false;
    public float timeTeam = 15;
    public bool beforeRoundTeam = true;
    [SyncVar] public bool DontAttack = false;
    public enum Selection { none, minato, saitama, goku, war };
    public Selection currentSelect;
    //  public GameObject MinatoButton;
    // public bool minato;
    public GameObject playerselected;
    public GameObject MinatoPlayer;
    //   public GameObject SaitamaButton;
    // public bool Saitama;
    public GameObject SaitamaPlayer;
    //   public GameObject GokuButton;
    //  public bool Goku;
    public GameObject GokuPlayer;
    //    public GameObject WarButton;
    // public bool War;
    public GameObject WarPlayer;
    public GameObject LobbyFrom;
    public bool airDodgeFall;
    public int team;
    bool heavyStuckColliderBool = false;
    bool runDodgeBool = false;
    bool chargeStuck = false;
    bool dodgeJumpGroundBool = false;
    bool dodgeJumpCancelCorou = false;
    bool dodgeJumpCanceled = false;
    bool dodgeJumpCancel = false;
    bool heavyWaitBool = false;
    bool WarHeavyStun = false;
    bool canHeavy = true;
    bool HeavyStuckBool = false;
    bool canFoot = false;
    bool afterHoldHeavy = false;
    float startTime = 0f;
    float holdTime = 0.3f;
    public AudioSource Slash;
    public Image imageCooldownAbility1;
    public Image imageCooldownAbility2;
    public Image imageCooldownAbility3;
    public Image imageCooldownAbility4;
    public Image imageCooldownAbility5;
    bool isCoolDown1;
    bool isCoolDown2;
    bool isCoolDown3;
    bool isCoolDown4;
    bool isCoolDown5;
    public bool canGreyFire = true;
    public bool canGaruru = true;
    public bool canGaia = true;
    float SpPerSecond = 3f;
    private float regenSp;
    float ammoPerSecond = 5f;
    private float regenAmmo;
    public RectTransform ammobar;
    [SyncVar(hook = "OnChangeAmmo")] public int currentAmmo = maxAmmo;
    public const int maxAmmo = 100;
    public RectTransform spbar;
    [SyncVar(hook = "OnChangeSp")] public int currentSp = maxSp;
    public const int maxSp = 100;
    bool one_z = false;
    bool timer_running;
    float timer_for_double_z;
    float delay = 0.5f;
    float BraveSpeed = 50;
    float moveSpeed = 8;
    float jumpForce = 20;
    float dodgeSpeed = 20;
    float fallForce = -10;
    public CharacterController controller;
    float gravityScale = 7;
    public Animator anim;
    public Transform pivot;
    float rotateSpeed = 30;
    float runSpeed = 15;
    float attackSpeed = 10;
    float omegaSpeed = 50;
    float clawSpeed = 30;
    float clawSpeed2 = 180;
    float wallSpeed = 30;
    public GameObject Camera1;
    public GameObject OmeCamera;
    public Camera Camera2;
    public Camera OmeCamera2;
    public Camera cam;
    public int CamMode;
    public AudioSource DramonKillerSound;
    public AudioSource SwordSound;
    Rigidbody  Rigid;
    bool action = false;
    float cooldowntime = 50;
    float cooldowntime2 = 40;
    float cooldowntime3 = 20;
    float cooldowntime4 = 15;
    float cooldowntime5 = 1;
    float cooldowntime6 = 1;
    float cooldowntime7 = 0.3f;
    float cooldowntime9 = 5;
    float cooldowntime10 = 40;
    float cooldowntime12 = 0.5f;
    float cooldowntime13 = 1f;
    float cooldowntime14 = 2f;
    float nextFireTime = 0;
    float nextFireTime2 = 0;
    float nextFireTime3 = 0;
    float nextFireTime4 = 0;
    float nextFireTime5 = 0;
    float nextFireTime6 = 0;
    float nextFireTime7 = 0;
    float nextFireTime8 = 0;
    float nextFireTime9 = 0;
    float nextFireTime10 = 0;
    float nextFireTime12 = 0;
    float nextFireTime13 = 0;
    float nextFireTime14 = 0;
    bool charge;
    bool Shot;
    public AudioSource Impact2;
    public Transform target;
    public bool canAttack = true;
    bool turning = false;
    bool pushed = false;
    bool WallHited = false;
    bool Down1 = false;
    bool Down2 = false;
    bool downWallFall = false;
    bool DownWall = false;
    bool Down3 = false;
    bool DashedUltra = false;
    public bool canDodge = false;
    private float knockBackCounter;
    public GameObject AudioListenerCamera;
    private Vector3 moveDirection;
    public GameObject playerModel;
    public Transform cameraTransform;
    public Transform cameraTransform2;
  
    void Start()
    { if (!isLocalPlayer)
        {
            AudioListenerCamera.SetActive(false);
            Camera2.enabled = false;
            OmeCamera2.enabled = false;
            Destroy(this);

            return;
        }
        controller = GetComponent<CharacterController>();
        ToPManager = GameObject.FindWithTag("ToPManager");
    }

There’s way more after

using System;
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using UnityEngine.Networking;
using UnityEngine.Networking.Match;

public class MyLobby : NetworkManager {
    public bool canDontAttackLaunch = false;
    public float TimeButtonJoin = 0f;
    public bool endBeforeTime = false;
    public bool newRound = false;
    public bool DontAttackManager = true;
    public bool DontStartRound = true;
    public bool StartRoundBool = true;
    public GameObject ToPManager;
    public GameObject WarpToGame;
    public float timeEndRound = 300f;
    public bool EndRoundBool = false;
    public float timeStartRound = 15f;
    public float timeStartRoundAttack = 15f;
    public bool StartAttackBool = false;
    public GameObject SpawnEnter;
    public GameObject LobbyOption;
    public GameObject TitleScreen;
    private bool TitleBool;
    public enum Selection { none, minato, saitama, goku, war };
    public Selection currentSelect;
    private float nextRefreshTime = 2f;
    //  public GameObject MinatoButton;
    // public bool minato;
    public GameObject playerselected;
    public GameObject MinatoPlayer;
//   public GameObject SaitamaButton;
   // public bool Saitama;
    public GameObject SaitamaPlayer;
//   public GameObject GokuButton;
  //  public bool Goku;
    public GameObject GokuPlayer;
//    public GameObject WarButton;
   // public bool War;
    public GameObject WarPlayer;


    public GameObject Player;
    public void Start()
    {

        RefreshMatches();
    }
    public void StartHosting()
    {
      
        StartMatchMaker();
        matchMaker.CreateMatch("Room" + TimeButtonJoin, 4, true, "", "", "", 0, 0, OnMatchCreated);
        DontStartRound = false;
        LobbyOption.SetActive(false);
        canDontAttackLaunch = true;
    }

    private void OnMatchCreated(bool success, string extendedInfo, MatchInfo responseData)
    {
        base.StartHost(responseData);

        RefreshMatches();
    }
    public void Update()
    {
        TimeButtonJoin += Time.deltaTime;
        ToPManager = GameObject.FindWithTag("ToPManager");

        WarpToGame = GameObject.FindWithTag("WarpToGame");
        if (DontAttackManager == true && canDontAttackLaunch == true)
        {
            ToPManager.GetComponent<ToPManager>().CmdDontAttack();
        }
        if (EndRoundBool == false)
        {
            timeEndRound = 300;
        }
        else
        {
            timeEndRound -= Time.deltaTime;
            ToPManager.GetComponent<ToPManager>().Timer3();
        }
        if (StartAttackBool == false)
        {
            timeStartRoundAttack = 15;
        }
        else
        {
            timeStartRoundAttack -= Time.deltaTime;
            ToPManager.GetComponent<ToPManager>().Timer2();
        }
        if (DontStartRound == true)
        {
            timeStartRound = 15;
        }
        else
        {
            ToPManager.GetComponent<ToPManager>().Timer1();
            ToPManager.GetComponent<ToPManager>().CmdTeamSelectMoment();
            timeStartRound -= Time.deltaTime;
        }

            if (timeStartRound < 0 & StartRoundBool == true)
        {
            ToPManager.GetComponent<ToPManager>().CmdStartRound();
            WarpToGame.GetComponent<WarpToGame>().canRejoin = true;
            WarpToGame.GetComponent<WarpToGame>().canRejoinSurvivor1 = true;
            WarpToGame.GetComponent<WarpToGame>().canRejoinSurvivor2 = true;
            DontAttackManager = true;
            DontStartRound = true;
            StartRoundBool = false;
            StartAttackBool = true;
        }
            if(timeStartRoundAttack < 0)
        {
            EndRoundBool = true;
            DontAttackManager = false;
            ToPManager.GetComponent<ToPManager>().CmdAttack();
            StartAttackBool = false;

        }
        if (timeEndRound < 0 || endBeforeTime == true)
        {
            EndRoundBool = false;
            DontAttackManager = true;
            newRound = true;
            endBeforeTime = false;
        }
        if(newRound == true)
        {
            StartRoundBool = true;
            WarpToGame.GetComponent<WarpToGame>().canRejoin = false;
            ToPManager.GetComponent<ToPManager>().EndRound();
            ToPManager.GetComponent<ToPManager>().CmdTeamWonTime();
            StartCoroutine(NewRound(10f));
            newRound = false;
        }

        nextRefreshTime -= Time.deltaTime;

        if (nextRefreshTime <= 0)
        {
          
            RefreshMatches();
            nextRefreshTime = 2;
        }
      
        if(currentSelect == Selection.war)
        {
            playerselected = WarPlayer;
        }
        if (currentSelect == Selection.minato)
        {
            playerselected = MinatoPlayer;
        }
        if (currentSelect == Selection.saitama)
        {
            playerselected = SaitamaPlayer;
        }
        if (currentSelect == Selection.goku)
        {
            playerselected = GokuPlayer;
        }
        // currentSelect = MinatoButton.GetComponent<CharacterSelectionScreen>().currentSelect;
        //  Goku = GokuButton.GetComponent<CharacterSelectionScreen>().GokuSelected;
        // Saitama = SaitamaButton.GetComponent<CharacterSelectionScreen>().SaitamaSelected;
        // War = WarButton.GetComponent<CharacterSelectionScreen>().WarSelected;
    }

    private void RefreshMatches()
    {
        if (matchMaker == null);
        StartMatchMaker();

        matchMaker.ListMatches(0, 10, "", true, 0, 0, HandleListMatchesComplete);

    }

    public void JoinMatch(MatchInfoSnapshot match)
    {
        if (matchMaker == null)
            StartMatchMaker();

        matchMaker.JoinMatch(match.networkId, "", "", "", 0, 0, HandleJoinedMatch);
        LobbyOption.SetActive(false);
        canDontAttackLaunch = true;
    }

    private void HandleJoinedMatch(bool success, string extendedInfo, MatchInfo responseData)
    {
        StartClient(responseData);
    }

    private void HandleListMatchesComplete(bool success, string extendedInfo, List<MatchInfoSnapshot> responseData)
    {
        AvaibleMatchesList.HandleNewMatchList(responseData);
    }
    public override void OnServerAddPlayer(NetworkConnection conn, short playerControllerId)
    {
     //   TitleScreen.SetActive(false);
     //  LobbyOption.SetActive(true);
            GameObject player = (GameObject)Instantiate(playerPrefab, SpawnEnter.transform.position, Quaternion.identity);
         
            NetworkServer.AddPlayerForConnection(conn, player, playerControllerId);

    }
    // if (team == 1)
    //    { player.GetComponent<MinatoController_Net>().team = 1; }
    //int team = TeamHandler.GetComponent<TeamHandler>().team;
 
  
    public void Wargreymon()
    {
        //  LobbySelect.GetComponent<MyLobby>().currentSelect = war;
        currentSelect = Selection.war; // changes currentSel to Up.

    }

    //  MinatoButton.GetComponent<CharacterSelectionScreen>().MinatoSelected = false;
    //    SaitamaButton.GetComponent<CharacterSelectionScreen>().SaitamaSelected = false;
    //   GokuButton.GetComponent<CharacterSelectionScreen>().GokuSelected = false;
    //   WarSelected = true;

    public void Minato()
    {
        currentSelect = Selection.minato; // changes currentSel to Up.
                                          //  WarButton.GetComponent<CharacterSelectionScreen>().WarSelected = false;
                                          //  SaitamaButton.GetComponent<CharacterSelectionScreen>().SaitamaSelected = false;
                                          //   GokuButton.GetComponent<CharacterSelectionScreen>().GokuSelected = false;
                                          //   MinatoSelected = true;
    }
    public void Goku()
    {
        currentSelect = Selection.goku; // changes currentSel to Up.
                                        //  WarButton.GetComponent<CharacterSelectionScreen>().WarSelected = false;      
                                        //   MinatoButton.GetComponent<CharacterSelectionScreen>().MinatoSelected = false;
                                        //   SaitamaButton.GetComponent<CharacterSelectionScreen>().SaitamaSelected = false;
                                        //   GokuSelected = true;
    }
    public void Saitama()
    {
        currentSelect = Selection.saitama; // changes currentSel to Up.
                                           //  WarButton.GetComponent<CharacterSelectionScreen>().WarSelected = false;
                                           //  MinatoButton.GetComponent<CharacterSelectionScreen>().MinatoSelected = false;
                                           ////  GokuButton.GetComponent<CharacterSelectionScreen>().GokuSelected = false;
                                           //SaitamaSelected = true;
    }
        public void PlayerChoosed(NetworkIdentity player)
        {

            var conn = player.connectionToClient;

            var newPlayer = Instantiate<GameObject>(playerselected);

       // LobbyOption.SetActive(false);
            Destroy(player.gameObject);
            NetworkServer.ReplacePlayerForConnection(conn, newPlayer, 0);
        }

    public IEnumerator NewRound(float waitTime)
    {
        yield return new WaitForSeconds(waitTime);
        DontStartRound = false;

    }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.Networking;

public class OriginalPlayerPrefab : NetworkBehaviour {
    public enum Selection { none, minato, saitama, goku, war };
    public Selection currentSelect;
    public GameObject playerselected;
    public GameObject MinatoPlayer;
    public GameObject SaitamaPlayer;
    public GameObject GokuPlayer;
    public GameObject WarPlayer;
    public GameObject SpawnPoint2;
    // Use this for initialization
    void Start ()
    {
        SpawnPoint2 = GameObject.FindWithTag("SpawnPoint2");

    }
   
    // Update is called once per frame
    void Update () {

        if (currentSelect == Selection.war)
        {
            playerselected = WarPlayer;
        }
        if (currentSelect == Selection.minato)
        {
            playerselected = MinatoPlayer;
        }
        if (currentSelect == Selection.saitama)
        {
            playerselected = SaitamaPlayer;
        }
        if (currentSelect == Selection.goku)
        {
            playerselected = GokuPlayer;
        }
        if (Input.GetButtonDown("WarButton"))
        {
            Wargreymon();
        }
        if (Input.GetButtonDown("MinatoButton"))
        {
            Minato();
        }
        if (Input.GetButtonDown("SwitchCharacter"))
        {
            CmdSpawn();
        }

    }
    [Command]
    public void CmdSpawn()
    {
        GameObject go = (GameObject)Instantiate(playerselected, SpawnPoint2.transform.position, Quaternion.identity);
        NetworkServer.SpawnWithClientAuthority(go, connectionToClient);
    }
    public void Wargreymon()
    {
        //  LobbySelect.GetComponent<MyLobby>().currentSelect = war;
        currentSelect = Selection.war; // changes currentSel to Up.

    }

    //  MinatoButton.GetComponent<CharacterSelectionScreen>().MinatoSelected = false;
    //    SaitamaButton.GetComponent<CharacterSelectionScreen>().SaitamaSelected = false;
    //   GokuButton.GetComponent<CharacterSelectionScreen>().GokuSelected = false;
    //   WarSelected = true;

    public void Minato()
    {
        currentSelect = Selection.minato; // changes currentSel to Up.
                                          //  WarButton.GetComponent<CharacterSelectionScreen>().WarSelected = false;
                                          //  SaitamaButton.GetComponent<CharacterSelectionScreen>().SaitamaSelected = false;
                                          //   GokuButton.GetComponent<CharacterSelectionScreen>().GokuSelected = false;
                                          //   MinatoSelected = true;
    }
    public void Goku()
    {
        currentSelect = Selection.goku; // changes currentSel to Up.
                                        //  WarButton.GetComponent<CharacterSelectionScreen>().WarSelected = false;       
                                        //   MinatoButton.GetComponent<CharacterSelectionScreen>().MinatoSelected = false;
                                        //   SaitamaButton.GetComponent<CharacterSelectionScreen>().SaitamaSelected = false;
                                        //   GokuSelected = true;
    }
    public void Saitama()
    {
        currentSelect = Selection.saitama; // changes currentSel to Up.
                                           //  WarButton.GetComponent<CharacterSelectionScreen>().WarSelected = false;
                                           //  MinatoButton.GetComponent<CharacterSelectionScreen>().MinatoSelected = false;
                                           ////  GokuButton.GetComponent<CharacterSelectionScreen>().GokuSelected = false;
                                           //SaitamaSelected = true;
    }
}

You think the local authority is assigned but my code got a problem?

7

Note that Unet was not designed with the flexibility of destroying NetworkBehaviour components at your whim in mind. You should expect issues syncing state for objects you are doing so.

On your actual problem, you check for client authority with hasAuthority, not isLocalPlayer. isLocalPlayer is to check only if this object is this client’s special Player GameObject.

Lastly, Unet is deprecated, if you are not 90+% done with your project you should be converting your game to an alternative networking API right now instead of figuring out how to use Unet. Unet is announced to be removed from Unity in either 2019.2 or 2019.3 (originally 2019.1, but the devs have said they are pushing it back a bit, but still its removal is just months away).

8

@Joe-Censored talking about that, what would you suggest? Wait till Unity’s Google Multiplay is released? Or just go with a 3rd party like Photon?

@tijanikun There’s a few things I’m confused with here. Firstly to spawn things with local player authority, you’d probably already want a local player there. Plus, when you’re hitting the Destroy() method, it’s not sending a message for that to happen on the server and the other clients, it’s just happening on the local client. But then again having a player would not cause creating an object with local player authority to fail. In fact you need a local player to be able to assign an object with local player authority. Does that make sense?

9

OriginalPrefabs = 1st thing spawned and localplayer Object
PlayerGameObject = 2nd thing spawned with Authority

Hi, im using hasAuthority now and the server can spawn the PlayerGameObject with the Authority, but only if i uncheck the localplayerauthority of the PlayerGameObject prefab on inspector, in that case i got a console error :
AssignClientAuthority can only be used for NetworkIdentity component with LocalPlayerAuthority set.
UnityEngine.Networking.NetworkServer:SpawnWithClientAuthority(GameObject, NetworkConnection)
If i keep check the localplayerautority the spawned PlayerGameObject appear but doesn’t have hasAuthority

Should i keep uncheck local and keep the error?
Note that i can’t spawn the player from client side, nothing happens, not even error message

Hi,
The local player is the OriginalPrefabs
Destroy() was on my script from a video, im guessing it’s works because with that, the other player can’t use this script because it disappear on their side because they are not localplayer (this script include mouvement) but now im using hasAuthority and i got other problem said before

Thank you to give me some of your time by the way

edit: i just noticed that when a client join the matchmaking room i got this error:
There is already a player at that playerControllerId for this connections.
UnityEngine.Networking.NetworkIdentity:UNetStaticUpdate()
Can that be an issue?

10

I found on google out that you can’t use if(!hasAuthority) on start method, so i made it on update and with some other adjustments, it works!

I couldn’t do it without your advise, thank you both, this forum has so nice people~

1 Like
11

TL;DR: I really have no confidence anymore in using any network library from anyone else. Write it yourself. Rant below

Well so far with the project I’m currently working on, I started it using 3rd party uLink. I put that project on the back burner partly because uLink got abandoned by its developers. A couple years later I rewrote the project from the beginning around Unet. Things were going good, but I estimate I spent about 3 months total of my spare time dealing with issues that were hard to investigate because they were from the black box part of Unet who’s source has not been made public. I have made a few threads on my issues at various times, and the Unity developer who wrote the Unet LLAPI has been as responsive and helpful as he can, but that doesn’t mean that bugs in code I cannot view suddenly do not exist.

My last “final straw” issue for example was a problem that over a month of off and on trying to track down I finally narrowed down to some problem with fragmented channels causing outbound messages to get stuck in the outbound queue for that connection, eventually filling and blocking all communication with that single client, with no means provided by Unity to either just flush the queue or even query its contents. If I was right I figured I could spend the next 2 weeks of my available dev time writing and testing my own fragmented message system, and recompile the HLAPI to use it (because I was sending fragmented messages using Commands, ClientRpc, etc). If I was wrong, since I couldn’t be actually sure my assessment was correct, I’d be out 2 more weeks dealing with a problem which was caused by depending on code I could not modify or even inspect. And who knows the next time something like that would come up down the road yet again?

So I estimated it would take me 2 months to just write my own network API on top of the socket class. I’d already spent more than that just trying to troubleshoot Unet as it was. So that’s what I decided to do. And since I had 18 months or so invested in my project around Unet, I would build in many Unet HLAPI like features to make my conversion easier. A week after I started on that Unity announced they were deprecating Unet, which didn’t surprise me. My 2 month estimate turned into 2 1/2, and another week to convert my project, but it has been going great ever since. When I hit a networking issue, now I not only can go to the code to see what is going on, but I know the thing inside and out. Probably the best decision I’ve ever made with this project.

Funny thing is Unity has been working on a new network implementation for what, a year now? All they really have to show for it is a no frills unreliable UDP transport wrapper (unreliable as in they don’t implement reliability on top of UDP even though most projects will need it for some data). In less than 3 months I was able to throw together a system with per channel optional reliability and weak encryption, automatic message fragmentation up to near max int bytes in size, object spawning/destroy, automatic distance based object updates (don’t get updates as often or at all from objects far from the player), basic syncvars and rpcs, and more.

Unity has done what? Implemented a connection oriented unreliable transport on connectionless UDP, and really no other functionality. Why is Unity taking so long with this stuff though, I thought they have a whole team working on this, but just working during lunch and at night I was able to get so much more together in so much less time? I’m not saying my code is perfect or anything, and I’m mid tier as a programmer at best, but what is going on with Unity’s stuff? Are they just focused on their Google server offering they can charge money for even though google’s cloud servers are really too expensive already for most games to use? I’m usually a Unity fan boy, but the longer their new networking API is taking to get even minimal features it is just getting embarrassing. And this is their 3rd networking implementation the engine will have in just a few years.

So long story short, I’d stop trying to use someone else’s network library. The Socket class is really not that hard to use, and you can build any higher level functionality you actually need on top of it that fits your project well. The time it takes you to write your own networking code is actually likely less time than you will spend troubleshooting your networking problems coming from someone else’s network library.