From 4bcff65ad08db0aba80505602ea2a4844b7c3ea4 Mon Sep 17 00:00:00 2001
From: Perttu Ahola <celeron55@gmail.com>
Date: Tue, 29 Nov 2011 22:23:03 +0200
Subject: [PATCH] Better version match fail messages from server

---
 src/server.cpp | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/src/server.cpp b/src/server.cpp
index 4cabcda0c..64e67c465 100644
--- a/src/server.cpp
+++ b/src/server.cpp
@@ -2069,8 +2069,11 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
 			infostream<<"Server: Cannot negotiate "
 					"serialization version with peer "
 					<<peer_id<<std::endl;
-			SendAccessDenied(m_con, peer_id,
-					L"Your client is too old (map format)");
+			SendAccessDenied(m_con, peer_id, std::wstring(
+					L"Your client's version is not supported.\n"
+					L"Server version is ")
+					+ narrow_to_wide(VERSION_STRING) + L"."
+			);
 			return;
 		}
 		
@@ -2088,18 +2091,23 @@ void Server::ProcessData(u8 *data, u32 datasize, u16 peer_id)
 
 		if(net_proto_version == 0)
 		{
-			SendAccessDenied(m_con, peer_id,
-					L"Your client is too old. Please upgrade.");
+			SendAccessDenied(m_con, peer_id, std::wstring(
+					L"Your client's version is not supported.\n"
+					L"Server version is ")
+					+ narrow_to_wide(VERSION_STRING) + L"."
+			);
 			return;
 		}
 		
-		/* Uhh... this should actually be a warning but let's do it like this */
 		if(g_settings->getBool("strict_protocol_version_checking"))
 		{
-			if(net_proto_version < PROTOCOL_VERSION)
+			if(net_proto_version != PROTOCOL_VERSION)
 			{
-				SendAccessDenied(m_con, peer_id,
-						L"Your client is too old. Please upgrade.");
+				SendAccessDenied(m_con, peer_id, std::wstring(
+						L"Your client's version is not supported.\n"
+						L"Server version is ")
+						+ narrow_to_wide(VERSION_STRING) + L"."
+				);
 				return;
 			}
 		}