search

MegaGlest / megaglest-masterserver

MegaGlest real-time strategy game master server
http://megaglest.org
12 stars 11 forks source link

Add new headless server tag #2

Closed tomreyn closed 5 years ago

tomreyn commented 7 years ago

Allow [H] tag for headless servers in addition to "Headless "

tomreyn commented 5 years ago

This will only get used if we make the game send this shorter "tag" instead. The whole protocol is still ugly (no authentication, plain text, HTTP GET), we need to come up with a proper protocol some day. ;-)

andy5995 commented 5 years ago

The shorter tag is sent from the current source in git, but not from 3.13.0, right?

As for protocol, did you have anything in mind already, @tomreyn ?

cc @sexybiggetje

tomreyn commented 5 years ago

So far, there is no code in the MegaGlest game (server) to send the shorter 'Headless' tag, yet. I don't write C, but I guess it should be easy to make the game send the '[H]' string instead of the '[Headless]' string it currently sends (if I remember this correctly).

If redone, the protocol the game and masterserver should talk should be some "RESTful" protocol. I started drafting this at https://docs.megaglest.org/MG/Master_Server a long time ago.

I am not going to design the protocol / API nor either side of the implementation, though. My motivation to spend time on MegaGlest is currently limited to maintaining services (hosting) in their current state. This said, I'll be very happy if anyone decided to contribute their time on coordinating this with the game developers (i.e. Softcoder + Titi) to make it part of the game, or (maybe easier, since both are rather busy and less motivated, too) developed a branch / fork improving on this.

andy5995 commented 5 years ago

So far, there is no code in the MegaGlest game (server) to send the shorter 'Headless' tag, yet. I don't write C, but I guess it should be easy to make the game send the '[H]' string instead of the '[Headless]' string it currently sends (if I remember this correctly).

@tomreyn I think that was already done in https://github.com/MegaGlest/megaglest-source/commit/4dc6b44e03994e31b5baaf4e8cc650bc585286fd#diff-1dcefe5eaa3e8f323c2fa6a56821e791

tomreyn commented 5 years ago

Ahem, you're right, this is to prove my bad memory. Thanks for pointing it out.

andy5995 commented 5 years ago

Don't sweat it, @tomreyn :) A one line patch from 2 years ago... I think pretty easy to forget. Cheers!