fix: prevent allocation of massive array (MirrorNetworking#2465)

* fix: prevent allocation of massive array

A poison message could contain an invalid length causing us to allocate massive arrays

* throw exception instead

* Update Assets/Mirror/Runtime/NetworkReader.cs

Co-authored-by: James Frowen <[email protected]>

* test for ReadArray

* swapping order of equation so that length doesnt overflow

* adding test case

* adding more tests for array length

* swapping order

* fixing expected message

Co-authored-by: James Frowen <[email protected]>

fix: Updated Basic example to work in Unity 2018.4

feat(SimpleWebTransport): adding option to allow for tls outside of t…

…he transport

fix: Added OnStopServer to NetworkMatchChecker

feat: allow types that inherit from Networkbehaviour to be used in sy…

…ncvar's (MirrorNetworking#2471)

* adding MakeGeneric method

* adding tests for NB syncvar+hook

* adding functions to NetworkBehaviour

* getting references to new functions

* fixing NB read so it always reads same number of bytes as write

* adding backing field and serialize for Nb Syncvar

* extra test

* adding ignore to transform test

* fixing test

perf(NetworkTransform): removing allocations from Command (MirrorNetw…

…orking#2491)

`byte[] payload` will cause a new array to be created when parameters are read, using `ArraySegment<byte>` avoids this.

fix: NetworkRoomManager.ReadyStatusChanged made public

fix: stopping values being reset before OnDestory is called (MirrorNe…

…tworking#2485)

* fix: stopping values being reset before OnDestory is called

* matches server side logic
* if an object is being fully destroyed it doesn't need its values to be reset

feat: Allow Networkbehaviours to be sent in rpc (MirrorNetworking#2478)

* adding read/write functions

* adding method to get WriteNetworkBehaviour

* adding methods to weavertypes

* adding method to get ReadNetworkBehaviour

* weaver test for NetworkBehaviour in RPC

* renaming var

* writing tests to make sure read/write works

* registering functions

* changing function and adding comment

* checking any writer is set

* adding Description to test

* collection test

* fixing call

* removing errors, networkbehaviour as read/write

* adding test to use the reader that weaver generates

* fixing import

* simplifying test

* removing null from spawned

* improving throw

* adding code coverage

* applying code suggestions

fix: NetworkRoomManager and NetworkRoomPlayer public fields

Made fields public again for inheritance.