Just bump the limit. 4K was pretty aggressive -- @Ripsn has proven you can
get beyond 8K in normal play, so to be safe, let's go to 128K.


On Sat, Jul 13, 2013 at 9:04 PM, Matthias Mailänder <
notifications@github.com> wrote:

> In OpenRA.Game/Server/Connection.cs:
>
> > @@ -86,6 +87,13 @@ public void ReadData(Server server)
> >  								ExpectLength = BitConverter.ToInt32(bytes, 0) - 4;
> >  								Frame = BitConverter.ToInt32(bytes, 4);
> >  								State = ReceiveState.Data;
> > +
> > +								if (ExpectLength < 0 || ExpectLength > MaxOrderLength)
> > +								{
> > +									server.DropClient(this);
> > +									Log.Write("server", "Dropping client {0} for excessive order length = {1}", PlayerIndex, ExpectLength);
>
> @Ripsn <https://github.com/Ripsn> was hosting and kicked himself when
> right-click spamming infantry units:
>
> Dropping client 0 for excessive order length = 8811
>
> This needs to be reverted as it is game breaking with the current order
> code.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/OpenRA/OpenRA/pull/3535/files#r5178900>
> .
>