repair Teeworlds server support

[illwieckz]

Teeworlds support is broken since many years. I have some experimental code on my computer but I faced a big problem:

• There is 2 way to query Teeworlds masters but each master that understand one of them only understand this one. The response format is the same for both queries. Both two are used today.
• There is 3 way to query Teeworlds servers but each server that understand one of them only understand this one. Both three are used today.
• The response format differs according to the way the server was queried, there is three way to query but there is only two response format. Hopefully, we can detect it without knowing how it was queried.
• I do not found a simple way in Qstat to do something like that:

Try send_tee_request_packet_1
If OK, deal_with_tee_packet
Else Try send_tee_request_packet_2
  If OK, deal_with_tee_packet
  Else Try send_tee_request_packet_3
  If OK, deal_with_tee_packet

[stevenh]

There's nothing to stop you using multiple queries with fall-back though them to ensure you get a response.

[illwieckz]

Yes, but I did not understand how to do that! :/

[illwieckz]

In tee.c I tried something like that:

int len_tee_statuspacket = 14;
int len_tee_statuspacke2 = 15;
int len_tee_statuspacke3 = 15;
char tee_statuspacket[14] = { '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', 'g', 'i', 'e', 'f' };
char tee_statuspacke2[15] = { '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', 'g', 'i', 'e', '2', '\x00' };
char tee_statuspacke3[15] = { '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', '\xFF', 'g', 'i', 'e', '3', '\x00' };

query_status_t send_tee_request_packet( struct qserver *server )
{
    query_status_t ret_packet;

    // Try first then second then third...

    ret_packet = send_packet( server, tee_statuspacket, len_tee_statuspacket );

    if (ret_packet < 0)
    {
        ret_packet = send_packet( server, tee_statuspacke2, len_tee_statuspacke2 );
    }

    if (ret_packet < 0)
    {
        ret_packet = send_packet( server, tee_statuspacke3, len_tee_statuspacke3 );
    }

    return ret_packet;
}

But it is certainly not the way to do it. I suppose it must be done elsewhere, in the function calling send_tee_request_packet for example.

[stevenh]

You could send all three packets like your doing then processes the return ensuring you pick the right decode.

[illwieckz]

OK, it needed only to do that:

- if (ret_packet < 0)
+ if (ret_packet == INPROGRESS)

:innocent:

[illwieckz]

You can try with

./qstat -tees 81.30.158.57:8299

or with:

./qstat -default tees -f /tmp/teelist.txt

where /tmp/teelist.cfg is http://pastebin.com/MALPwk7k