[quake3-bugzilla] [Bug 5099] Add support for dpmaster protocol

[bugzilla-daemon at icculus.org]

https://bugzilla.icculus.org/show_bug.cgi?id=5099

--- Comment #12 from Thilo Schulz <arny at ats.s.bawue.de> 2011-07-27 17:14:05 EDT ---
(In reply to comment #11)
> Just to be clear, all servers are stored in one big list. The Q3A servers are
> not different, they are just using "Quake3Arena" as their "gamename". In other
> words, there's no difference between declaring a server using the Q3A master
> protocol, and declaring it using the DP master protocol with "Quake3Arena" as
> its "gamename".

Yes.

> I wouldn't really see the point of removing the vanilla Q3A master protocol. It
> wouldn't simplify the code significantly, and IOQ3 would lose interoperability
> with other tools and master servers.

It would only lose interoperability with old master servers, not any server
browser tools. And if id updates to a more recent version of dpmaster (or maybe
id's server even already supports the getserversExt query?) even this point
would be moot.

> Anyway, the point is: indeed, IOQ3 itself doesn't need the DP protocol. But if
> a standalone game based on IOQ3 doesn't want to receive IP addresses of Q3
> servers from the masters (and doesn't want to change the network protocol
> number just for that purpose), it will need to send a custom gamename. And the
> only way to do that is to use the DP master protocol.

As said above, the only master server still making problems would be the one
above from id, and standalone games really shouldn't use that server. Are there
even alternatives to dpmaster? Wouldn't standalone games use your server
anyways?
I simply don't see the need for a "dpmasterprotocol" cvar in this case. Why not
always use getserversExt query in the standalone case, even when going over
ipv4? As has been established it is more than sufficient in my view to provide
the game separation you strive for, without adding yet more code.

> If it's not already in the IOQ3 code base, standalone games will have to patch
> the code themselves. So the idea here is to put the code as an option into the
> IOQ3 trunk to avoid redundant work.

Your server is a nice piece of work. I was just looking at the many changes the
patch introduces and I don't see the necessity in it because you did this nice
work. I stumbled over the gamename vs. heartbeat problems when adding AAAA
records to my master server for wop and realized the getserversExt query can be
used perfectly to separate games without needing to resort to stuff such as
parsing infostrings.

-- 
Configure bugmail: https://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.