summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Lutz <michi@icosahedron.de>2021-12-19 15:22:40 +0100
committerMichael Lutz <michi@icosahedron.de>2021-12-20 19:30:27 +0100
commite67124e1745737853aa26597aacdb36213da23d5 (patch)
tree1e19ecc8e62a14d91e11360e50a0aa11642cb55d
parent3fede159409bdddf73bfceec2145d8d419389486 (diff)
downloadopenttd-e67124e1745737853aa26597aacdb36213da23d5.tar.xz
Change: [Admin] Bump admin port protocol due to command changes.
docs/admin_network.md promised that information in an admin packet is never removed. It does allow the possibility of using a new packet type for changed data in combination with a bump of the admin port version. As the recent command handling changes modified the contents of ADMIN_PACKET_SERVER_CMD_LOGGING, do exactly that.
-rw-r--r--src/network/core/config.h2
-rw-r--r--src/network/core/tcp_admin.h3
2 files changed, 3 insertions, 2 deletions
diff --git a/src/network/core/config.h b/src/network/core/config.h
index a7493e817..3c9e1a50c 100644
--- a/src/network/core/config.h
+++ b/src/network/core/config.h
@@ -47,7 +47,7 @@ static const uint16 UDP_MTU = 1460; ///< Numbe
static const uint16 TCP_MTU = 32767; ///< Number of bytes we can pack in a single TCP packet
static const uint16 COMPAT_MTU = 1460; ///< Number of bytes we can pack in a single packet for backward compatibility
-static const byte NETWORK_GAME_ADMIN_VERSION = 1; ///< What version of the admin network do we use?
+static const byte NETWORK_GAME_ADMIN_VERSION = 2; ///< What version of the admin network do we use?
static const byte NETWORK_GAME_INFO_VERSION = 6; ///< What version of game-info do we use?
static const byte NETWORK_COMPANY_INFO_VERSION = 6; ///< What version of company info is this?
static const byte NETWORK_COORDINATOR_VERSION = 6; ///< What version of game-coordinator-protocol do we use?
diff --git a/src/network/core/tcp_admin.h b/src/network/core/tcp_admin.h
index 33ab7b049..97af6f0f4 100644
--- a/src/network/core/tcp_admin.h
+++ b/src/network/core/tcp_admin.h
@@ -56,10 +56,11 @@ enum PacketAdminType {
ADMIN_PACKET_SERVER_RCON, ///< The server's reply to a remove console command.
ADMIN_PACKET_SERVER_CONSOLE, ///< The server gives the admin the data that got printed to its console.
ADMIN_PACKET_SERVER_CMD_NAMES, ///< The server sends out the names of the DoCommands to the admins.
- ADMIN_PACKET_SERVER_CMD_LOGGING, ///< The server gives the admin copies of incoming command packets.
+ ADMIN_PACKET_SERVER_CMD_LOGGING_OLD, ///< Used to be the type ID of \c ADMIN_PACKET_SERVER_CMD_LOGGING in \c NETWORK_GAME_ADMIN_VERSION 1.
ADMIN_PACKET_SERVER_GAMESCRIPT, ///< The server gives the admin information from the GameScript in JSON.
ADMIN_PACKET_SERVER_RCON_END, ///< The server indicates that the remote console command has completed.
ADMIN_PACKET_SERVER_PONG, ///< The server replies to a ping request from the admin.
+ ADMIN_PACKET_SERVER_CMD_LOGGING, ///< The server gives the admin copies of incoming command packets.
INVALID_ADMIN_PACKET = 0xFF, ///< An invalid marker for admin packets.
};