andrewphorn / ClassiCube-Client

The applet used for classicube.net
34 stars 19 forks source link

New Loading behavior breaks function #291

Closed voidpublic closed 9 years ago

voidpublic commented 9 years ago

So basically with something in the new update /ref is no longer working, meaning its crashing the client completely

What it basically does is it reloads everything for the user (so sends MOTD) so i suppose that is the issue.

123DMWM commented 9 years ago

That sounds like a server problem. What server software is this on? Please follow This quide to help us.

mstefarov commented 9 years ago

I'll take a look. I've noticed that WoM-style extensions (via &cfg=... in motd) stopped working on my server too. Could be related to your problem.

voidpublic commented 9 years ago

Well i am based on MCForge but use my own distri, i mostly code it myself xwom / wom doesnt seem to have a problem:

    public void SendUserMOTD()
    {
        byte[] buffer = new byte[130];
        buffer[0] = Server.version;
        if (UsingWom && (level.textures.enabled || level.motd == "texture") && group.Permission >= level.textures.LowestRank.Permission) { StringFormat(Server.name, 64).CopyTo(buffer, 1); StringFormat("&0cfg=" + Server.IP + ":" + Server.port + "/" + level.name, 64).CopyTo(buffer, 65); }
        if (level.motd == "ignore")
        {
            StringFormat(Server.name, 64).CopyTo(buffer, 1);
            if (!String.IsNullOrEmpty(group.MOTD)) StringFormat(group.MOTD, 64).CopyTo(buffer, 65);
            else StringFormat(Server.motd, 64).CopyTo(buffer, 65);
        }
        else StringFormat(level.motd, 128).CopyTo(buffer, 1);
        //if (Block.canPlace(this.group.Permission, Block.blackrock))
            if (this.referee)
            buffer[129] = 100;
        else
            buffer[129] = 0;
        SendRaw(0, buffer);
    }

This after a player is already connected makes them crash.

I ruled out the motd, so its the SendRaw giving the Client problems

mstefarov commented 9 years ago

Reproduced. I'll be working on a fix. Thanks for reporting!

voidpublic commented 9 years ago

This should be included asap as this is degrading functionality

mstefarov commented 9 years ago

Almost done with all the bugfixes. Patch soon!

voidpublic commented 9 years ago

Appearently the patch didnt fix the issue if it was included in the last update?

123DMWM commented 9 years ago

Got a server up for testing?

mstefarov commented 9 years ago

WoM configs do load, but not all functionality is restored yet.

These parameters work:

These do not [yet]:

A separate issue (#305) has been created to track support for "environment" parameters.

mstefarov commented 9 years ago

Also, at no point did I run into client crashes while working on this. Even when client did not read configs, it did not crash. If you ARE experiencing a crash, you'll have to give us more details (e.g. what config you are sending, and when are you sending it).

voidpublic commented 9 years ago

https://gist.github.com/cec1266a92655909ac1d Debug mode doesnt really give me much, if you need a server where to reproduce, i can offer that, got any IM like skype? MOTD: Play fair, have fun -hax +ophax

andrewphorn commented 9 years ago

You can contact us on IRC, we're always available there.