Exposing the zoom key to Lua API (#9903)
[oweals/minetest.git] / src / network / serverpackethandler.cpp
index e6b400750f7afba206898a1c1c7b5b1d5e18e498..b3008bb50c22b31ba3f797a454850da111c00a36 100644 (file)
@@ -20,9 +20,9 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "chatmessage.h"
 #include "server.h"
 #include "log.h"
-#include "content_sao.h"
 #include "emerge.h"
 #include "mapblock.h"
+#include "modchannels.h"
 #include "nodedef.h"
 #include "remoteplayer.h"
 #include "rollback_interface.h"
@@ -30,8 +30,11 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "settings.h"
 #include "tool.h"
 #include "version.h"
+#include "network/connection.h"
 #include "network/networkprotocol.h"
 #include "network/serveropcodes.h"
+#include "server/player_sao.h"
+#include "server/serverinventorymgr.h"
 #include "util/auth.h"
 #include "util/base64.h"
 #include "util/pointedthing.h"
@@ -50,11 +53,12 @@ void Server::handleCommand_Init(NetworkPacket* pkt)
        if(pkt->getSize() < 1)
                return;
 
-       RemoteClient* client = getClient(pkt->getPeerId(), CS_Created);
+       session_t peer_id = pkt->getPeerId();
+       RemoteClient *client = getClient(peer_id, CS_Created);
 
        std::string addr_s;
        try {
-               Address address = getPeerAddress(pkt->getPeerId());
+               Address address = getPeerAddress(peer_id);
                addr_s = address.serializeString();
        }
        catch (con::PeerNotFoundException &e) {
@@ -64,27 +68,27 @@ void Server::handleCommand_Init(NetworkPacket* pkt)
                 * respond for some time, your server was overloaded or
                 * things like that.
                 */
-               infostream << "Server::ProcessData(): Canceling: peer "
-                               << pkt->getPeerId() << " not found" << std::endl;
+               infostream << "Server::ProcessData(): Canceling: peer " << peer_id <<
+                       " not found" << std::endl;
                return;
        }
 
        // If net_proto_version is set, this client has already been handled
        if (client->getState() > CS_Created) {
-               verbosestream << "Server: Ignoring multiple TOSERVER_INITs from "
-                               << addr_s << " (peer_id=" << pkt->getPeerId() << ")" << std::endl;
+               verbosestream << "Server: Ignoring multiple TOSERVER_INITs from " <<
+                       addr_s << " (peer_id=" << peer_id << ")" << std::endl;
                return;
        }
 
-       verbosestream << "Server: Got TOSERVER_INIT from " << addr_s << " (peer_id="
-                       << pkt->getPeerId() << ")" << std::endl;
+       verbosestream << "Server: Got TOSERVER_INIT from " << addr_s <<
+               " (peer_id=" << peer_id << ")" << std::endl;
 
        // Do not allow multiple players in simple singleplayer mode.
        // This isn't a perfect way to do it, but will suffice for now
        if (m_simple_singleplayer_mode && m_clients.getClientIDs().size() > 1) {
-               infostream << "Server: Not allowing another client (" << addr_s
-                               << ") to connect in simple singleplayer mode" << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_SINGLEPLAYER);
+               infostream << "Server: Not allowing another client (" << addr_s <<
+                       ") to connect in simple singleplayer mode" << std::endl;
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_SINGLEPLAYER);
                return;
        }
 
@@ -107,11 +111,9 @@ void Server::handleCommand_Init(NetworkPacket* pkt)
                depl_serial_v = SER_FMT_VER_INVALID;
 
        if (depl_serial_v == SER_FMT_VER_INVALID) {
-               actionstream << "Server: A mismatched client tried to connect from "
-                               << addr_s << std::endl;
-               infostream<<"Server: Cannot negotiate serialization version with "
-                               << addr_s << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_WRONG_VERSION);
+               actionstream << "Server: A mismatched client tried to connect from " <<
+                       addr_s << " ser_fmt_max=" << (int)client_max << std::endl;
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_WRONG_VERSION);
                return;
        }
 
@@ -140,25 +142,16 @@ void Server::handleCommand_Init(NetworkPacket* pkt)
 
        client->net_proto_version = net_proto_version;
 
-       // On this handler at least protocol version 25 is required
-       if (net_proto_version < 25 ||
+       if ((g_settings->getBool("strict_protocol_version_checking") &&
+                       net_proto_version != LATEST_PROTOCOL_VERSION) ||
                        net_proto_version < SERVER_PROTOCOL_VERSION_MIN ||
                        net_proto_version > SERVER_PROTOCOL_VERSION_MAX) {
-               actionstream << "Server: A mismatched client tried to connect from "
-                               << addr_s << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_WRONG_VERSION);
+               actionstream << "Server: A mismatched client tried to connect from " <<
+                       addr_s << " proto_max=" << (int)max_net_proto_version << std::endl;
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_WRONG_VERSION);
                return;
        }
 
-       if (g_settings->getBool("strict_protocol_version_checking")) {
-               if (net_proto_version != LATEST_PROTOCOL_VERSION) {
-                       actionstream << "Server: A mismatched (strict) client tried to "
-                                       << "connect from " << addr_s << std::endl;
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_WRONG_VERSION);
-                       return;
-               }
-       }
-
        /*
                Validate player name
        */
@@ -166,60 +159,56 @@ void Server::handleCommand_Init(NetworkPacket* pkt)
 
        size_t pns = playerName.size();
        if (pns == 0 || pns > PLAYERNAME_SIZE) {
-               actionstream << "Server: Player with "
-                       << ((pns > PLAYERNAME_SIZE) ? "a too long" : "an empty")
-                       << " name tried to connect from " << addr_s << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_WRONG_NAME);
+               actionstream << "Server: Player with " <<
+                       ((pns > PLAYERNAME_SIZE) ? "a too long" : "an empty") <<
+                       " name tried to connect from " << addr_s << std::endl;
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_WRONG_NAME);
                return;
        }
 
-       if (string_allowed(playerName, PLAYERNAME_ALLOWED_CHARS) == false) {
-               actionstream << "Server: Player with an invalid name "
-                               << "tried to connect from " << addr_s << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_WRONG_CHARS_IN_NAME);
+       if (!string_allowed(playerName, PLAYERNAME_ALLOWED_CHARS)) {
+               actionstream << "Server: Player with an invalid name tried to connect "
+                       "from " << addr_s << std::endl;
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_WRONG_CHARS_IN_NAME);
                return;
        }
 
-       m_clients.setPlayerName(pkt->getPeerId(), playername);
+       m_clients.setPlayerName(peer_id, playername);
        //TODO (later) case insensitivity
 
        std::string legacyPlayerNameCasing = playerName;
 
        if (!isSingleplayer() && strcasecmp(playername, "singleplayer") == 0) {
-               actionstream << "Server: Player with the name \"singleplayer\" "
-                               << "tried to connect from " << addr_s << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_WRONG_NAME);
+               actionstream << "Server: Player with the name \"singleplayer\" tried "
+                       "to connect from " << addr_s << std::endl;
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_WRONG_NAME);
                return;
        }
 
        {
                std::string reason;
                if (m_script->on_prejoinplayer(playername, addr_s, &reason)) {
-                       actionstream << "Server: Player with the name \"" << playerName << "\" "
-                                       << "tried to connect from " << addr_s << " "
-                                       << "but it was disallowed for the following reason: "
-                                       << reason << std::endl;
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_CUSTOM_STRING,
-                                       reason.c_str());
+                       actionstream << "Server: Player with the name \"" << playerName <<
+                               "\" tried to connect from " << addr_s <<
+                               " but it was disallowed for the following reason: " << reason <<
+                               std::endl;
+                       DenyAccess(peer_id, SERVER_ACCESSDENIED_CUSTOM_STRING, reason);
                        return;
                }
        }
 
-       infostream << "Server: New connection: \"" << playerName << "\" from "
-                       << addr_s << " (peer_id=" << pkt->getPeerId() << ")" << std::endl;
+       infostream << "Server: New connection: \"" << playerName << "\" from " <<
+               addr_s << " (peer_id=" << peer_id << ")" << std::endl;
 
        // Enforce user limit.
-       // Don't enforce for users that have some admin right
+       // Don't enforce for users that have some admin right or mod permits it.
        if (m_clients.isUserLimitReached() &&
-                       !checkPriv(playername, "server") &&
-                       !checkPriv(playername, "ban") &&
-                       !checkPriv(playername, "privs") &&
-                       !checkPriv(playername, "password") &&
-                       playername != g_settings->get("name")) {
-               actionstream << "Server: " << playername << " tried to join from "
-                               << addr_s << ", but there" << " are already max_users="
-                               << g_settings->getU16("max_users") << " players." << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_TOO_MANY_USERS);
+                       playername != g_settings->get("name") &&
+                       !m_script->can_bypass_userlimit(playername, addr_s)) {
+               actionstream << "Server: " << playername << " tried to join from " <<
+                       addr_s << ", but there are already max_users=" <<
+                       g_settings->getU16("max_users") << " players." << std::endl;
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_TOO_MANY_USERS);
                return;
        }
 
@@ -239,20 +228,19 @@ void Server::handleCommand_Init(NetworkPacket* pkt)
                                auth_mechs |= AUTH_MECHANISM_SRP;
                                client->enc_pwd = encpwd;
                        } else {
-                               actionstream << "User " << playername
-                                       << " tried to log in, but password field"
-                                       << " was invalid (unknown mechcode)." << std::endl;
-                               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_SERVER_FAIL);
+                               actionstream << "User " << playername << " tried to log in, "
+                                       "but password field was invalid (unknown mechcode)." <<
+                                       std::endl;
+                               DenyAccess(peer_id, SERVER_ACCESSDENIED_SERVER_FAIL);
                                return;
                        }
                } else if (base64_is_valid(encpwd)) {
                        auth_mechs |= AUTH_MECHANISM_LEGACY_PASSWORD;
                        client->enc_pwd = encpwd;
                } else {
-                       actionstream << "User " << playername
-                               << " tried to log in, but password field"
-                               << " was invalid (invalid base64)." << std::endl;
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_SERVER_FAIL);
+                       actionstream << "User " << playername << " tried to log in, but "
+                               "password field was invalid (invalid base64)." << std::endl;
+                       DenyAccess(peer_id, SERVER_ACCESSDENIED_SERVER_FAIL);
                        return;
                }
        } else {
@@ -275,8 +263,8 @@ void Server::handleCommand_Init(NetworkPacket* pkt)
        verbosestream << "Sending TOCLIENT_HELLO with auth method field: "
                << auth_mechs << std::endl;
 
-       NetworkPacket resp_pkt(TOCLIENT_HELLO, 1 + 4
-               + legacyPlayerNameCasing.size(), pkt->getPeerId());
+       NetworkPacket resp_pkt(TOCLIENT_HELLO,
+               1 + 4 + legacyPlayerNameCasing.size(), peer_id);
 
        u16 depl_compress_mode = NETPROTO_COMPRESSION_NONE;
        resp_pkt << depl_serial_v << depl_compress_mode << net_proto_version
@@ -287,369 +275,69 @@ void Server::handleCommand_Init(NetworkPacket* pkt)
        client->allowed_auth_mechs = auth_mechs;
        client->setDeployedCompressionMode(depl_compress_mode);
 
-       m_clients.event(pkt->getPeerId(), CSE_Hello);
+       m_clients.event(peer_id, CSE_Hello);
 }
 
-void Server::handleCommand_Init_Legacy(NetworkPacket* pkt)
+void Server::handleCommand_Init2(NetworkPacket* pkt)
 {
-       // [0] u8 SER_FMT_VER_HIGHEST_READ
-       // [1] u8[20] player_name
-       // [21] u8[28] password <--- can be sent without this, from old versions
-
-       if (pkt->getSize() < 1+PLAYERNAME_SIZE)
-               return;
-
-       RemoteClient* client = getClient(pkt->getPeerId(), CS_Created);
-
-       std::string addr_s;
-       try {
-               Address address = getPeerAddress(pkt->getPeerId());
-               addr_s = address.serializeString();
-       }
-       catch (con::PeerNotFoundException &e) {
-               /*
-                * no peer for this packet found
-                * most common reason is peer timeout, e.g. peer didn't
-                * respond for some time, your server was overloaded or
-                * things like that.
-                */
-               infostream << "Server::ProcessData(): Canceling: peer "
-                               << pkt->getPeerId() << " not found" << std::endl;
-               return;
-       }
-
-       // If net_proto_version is set, this client has already been handled
-       if (client->getState() > CS_Created) {
-               verbosestream << "Server: Ignoring multiple TOSERVER_INITs from "
-                               << addr_s << " (peer_id=" << pkt->getPeerId() << ")" << std::endl;
-               return;
-       }
+       session_t peer_id = pkt->getPeerId();
+       verbosestream << "Server: Got TOSERVER_INIT2 from " << peer_id << std::endl;
 
-       verbosestream << "Server: Got TOSERVER_INIT_LEGACY from " << addr_s << " (peer_id="
-                       << pkt->getPeerId() << ")" << std::endl;
+       m_clients.event(peer_id, CSE_GotInit2);
+       u16 protocol_version = m_clients.getProtocolVersion(peer_id);
 
-       // Do not allow multiple players in simple singleplayer mode.
-       // This isn't a perfect way to do it, but will suffice for now
-       if (m_simple_singleplayer_mode && m_clients.getClientIDs().size() > 1) {
-               infostream << "Server: Not allowing another client (" << addr_s
-                               << ") to connect in simple singleplayer mode" << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Running in simple singleplayer mode.");
-               return;
-       }
-
-       // First byte after command is maximum supported
-       // serialization version
-       u8 client_max;
-
-       *pkt >> client_max;
-
-       u8 our_max = SER_FMT_VER_HIGHEST_READ;
-       // Use the highest version supported by both
-       int deployed = std::min(client_max, our_max);
-       // If it's lower than the lowest supported, give up.
-       if (deployed < SER_FMT_VER_LOWEST_READ)
-               deployed = SER_FMT_VER_INVALID;
-
-       if (deployed == SER_FMT_VER_INVALID) {
-               actionstream << "Server: A mismatched client tried to connect from "
-                               << addr_s << std::endl;
-               infostream<<"Server: Cannot negotiate serialization version with "
-                               << addr_s << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), std::wstring(
-                               L"Your client's version is not supported.\n"
-                               L"Server version is ")
-                               + utf8_to_wide(g_version_string) + L"."
-               );
-               return;
-       }
-
-       client->setPendingSerializationVersion(deployed);
+       std::string lang;
+       if (pkt->getSize() > 0)
+               *pkt >> lang;
 
        /*
-               Read and check network protocol version
+               Send some initialization data
        */
 
-       u16 min_net_proto_version = 0;
-       if (pkt->getSize() >= 1 + PLAYERNAME_SIZE + PASSWORD_SIZE + 2)
-               min_net_proto_version = pkt->getU16(1 + PLAYERNAME_SIZE + PASSWORD_SIZE);
-
-       // Use same version as minimum and maximum if maximum version field
-       // doesn't exist (backwards compatibility)
-       u16 max_net_proto_version = min_net_proto_version;
-       if (pkt->getSize() >= 1 + PLAYERNAME_SIZE + PASSWORD_SIZE + 2 + 2)
-               max_net_proto_version = pkt->getU16(1 + PLAYERNAME_SIZE + PASSWORD_SIZE + 2);
+       infostream << "Server: Sending content to " << getPlayerName(peer_id) <<
+               std::endl;
 
-       // Start with client's maximum version
-       u16 net_proto_version = max_net_proto_version;
-
-       // Figure out a working version if it is possible at all
-       if (max_net_proto_version >= SERVER_PROTOCOL_VERSION_MIN ||
-                       min_net_proto_version <= SERVER_PROTOCOL_VERSION_MAX) {
-               // If maximum is larger than our maximum, go with our maximum
-               if (max_net_proto_version > SERVER_PROTOCOL_VERSION_MAX)
-                       net_proto_version = SERVER_PROTOCOL_VERSION_MAX;
-               // Else go with client's maximum
-               else
-                       net_proto_version = max_net_proto_version;
-       }
-
-       // The client will send up to date init packet, ignore this one
-       if (net_proto_version >= 25)
-               return;
-
-       verbosestream << "Server: " << addr_s << ": Protocol version: min: "
-                       << min_net_proto_version << ", max: " << max_net_proto_version
-                       << ", chosen: " << net_proto_version << std::endl;
-
-       client->net_proto_version = net_proto_version;
-
-       if (net_proto_version < SERVER_PROTOCOL_VERSION_MIN ||
-                       net_proto_version > SERVER_PROTOCOL_VERSION_MAX) {
-               actionstream << "Server: A mismatched client tried to connect from "
-                               << addr_s << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), std::wstring(
-                               L"Your client's version is not supported.\n"
-                               L"Server version is ")
-                               + utf8_to_wide(g_version_string) + L",\n"
-                               + L"server's PROTOCOL_VERSION is "
-                               + utf8_to_wide(itos(SERVER_PROTOCOL_VERSION_MIN))
-                               + L"..."
-                               + utf8_to_wide(itos(SERVER_PROTOCOL_VERSION_MAX))
-                               + L", client's PROTOCOL_VERSION is "
-                               + utf8_to_wide(itos(min_net_proto_version))
-                               + L"..."
-                               + utf8_to_wide(itos(max_net_proto_version))
-               );
-               return;
-       }
-
-       if (g_settings->getBool("strict_protocol_version_checking")) {
-               if (net_proto_version != LATEST_PROTOCOL_VERSION) {
-                       actionstream << "Server: A mismatched (strict) client tried to "
-                                       << "connect from " << addr_s << std::endl;
-                       DenyAccess_Legacy(pkt->getPeerId(), std::wstring(
-                                       L"Your client's version is not supported.\n"
-                                       L"Server version is ")
-                                       + utf8_to_wide(g_version_string) + L",\n"
-                                       + L"server's PROTOCOL_VERSION (strict) is "
-                                       + utf8_to_wide(itos(LATEST_PROTOCOL_VERSION))
-                                       + L", client's PROTOCOL_VERSION is "
-                                       + utf8_to_wide(itos(min_net_proto_version))
-                                       + L"..."
-                                       + utf8_to_wide(itos(max_net_proto_version))
-                       );
-                       return;
-               }
-       }
+       // Send item definitions
+       SendItemDef(peer_id, m_itemdef, protocol_version);
 
-       /*
-               Set up player
-       */
-       char playername[PLAYERNAME_SIZE];
-       unsigned int playername_length = 0;
-       for (; playername_length < PLAYERNAME_SIZE; playername_length++ ) {
-               playername[playername_length] = pkt->getChar(1+playername_length);
-               if (pkt->getChar(1+playername_length) == 0)
-                       break;
-       }
-
-       if (playername_length == PLAYERNAME_SIZE) {
-               actionstream << "Server: Player with name exceeding max length "
-                               << "tried to connect from " << addr_s << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Name too long");
-               return;
-       }
+       // Send node definitions
+       SendNodeDef(peer_id, m_nodedef, protocol_version);
 
+       m_clients.event(peer_id, CSE_SetDefinitionsSent);
 
-       if (playername[0]=='\0') {
-               actionstream << "Server: Player with an empty name "
-                               << "tried to connect from " << addr_s << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Empty name");
-               return;
-       }
+       // Send media announcement
+       sendMediaAnnouncement(peer_id, lang);
 
-       if (string_allowed(playername, PLAYERNAME_ALLOWED_CHARS) == false) {
-               actionstream << "Server: Player with an invalid name "
-                               << "tried to connect from " << addr_s << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Name contains unallowed characters");
-               return;
-       }
+       RemoteClient *client = getClient(peer_id, CS_InitDone);
 
-       if (!isSingleplayer() && strcasecmp(playername, "singleplayer") == 0) {
-               actionstream << "Server: Player with the name \"singleplayer\" "
-                               << "tried to connect from " << addr_s << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Name is not allowed");
-               return;
-       }
+       // Keep client language for server translations
+       client->setLangCode(lang);
 
+       // Send active objects
        {
-               std::string reason;
-               if (m_script->on_prejoinplayer(playername, addr_s, &reason)) {
-                       actionstream << "Server: Player with the name \"" << playername << "\" "
-                                       << "tried to connect from " << addr_s << " "
-                                       << "but it was disallowed for the following reason: "
-                                       << reason << std::endl;
-                       DenyAccess_Legacy(pkt->getPeerId(), utf8_to_wide(reason.c_str()));
-                       return;
-               }
-       }
-
-       infostream<<"Server: New connection: \""<<playername<<"\" from "
-                       <<addr_s<<" (peer_id="<<pkt->getPeerId()<<")"<<std::endl;
-
-       // Get password
-       char given_password[PASSWORD_SIZE];
-       if (pkt->getSize() < 1 + PLAYERNAME_SIZE + PASSWORD_SIZE) {
-               // old version - assume blank password
-               given_password[0] = 0;
-       }
-       else {
-               for (u16 i = 0; i < PASSWORD_SIZE - 1; i++) {
-                       given_password[i] = pkt->getChar(21 + i);
-               }
-               given_password[PASSWORD_SIZE - 1] = 0;
-       }
-
-       if (!base64_is_valid(given_password)) {
-               actionstream << "Server: " << playername
-                               << " supplied invalid password hash" << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Invalid password hash");
-               return;
-       }
-
-       // Enforce user limit.
-       // Don't enforce for users that have some admin right
-       if (m_clients.isUserLimitReached() &&
-                       !checkPriv(playername, "server") &&
-                       !checkPriv(playername, "ban") &&
-                       !checkPriv(playername, "privs") &&
-                       !checkPriv(playername, "password") &&
-                       playername != g_settings->get("name")) {
-               actionstream << "Server: " << playername << " tried to join, but there"
-                               << " are already max_users="
-                               << g_settings->getU16("max_users") << " players." << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Too many users.");
-               return;
-       }
-
-       std::string checkpwd; // Password hash to check against
-       bool has_auth = m_script->getAuth(playername, &checkpwd, NULL);
-
-       // If no authentication info exists for user, create it
-       if (!has_auth) {
-               if (!isSingleplayer() &&
-                               g_settings->getBool("disallow_empty_password") &&
-                               std::string(given_password) == "") {
-                       actionstream << "Server: " << playername
-                                       << " supplied empty password" << std::endl;
-                       DenyAccess_Legacy(pkt->getPeerId(), L"Empty passwords are "
-                                       L"disallowed. Set a password and try again.");
-                       return;
-               }
-               std::string raw_default_password =
-                       g_settings->get("default_password");
-               std::string initial_password =
-                       translate_password(playername, raw_default_password);
-
-               // If default_password is empty, allow any initial password
-               if (raw_default_password.length() == 0)
-                       initial_password = given_password;
-
-               m_script->createAuth(playername, initial_password);
+               PlayerSAO *sao = getPlayerSAO(peer_id);
+               if (client && sao)
+                       SendActiveObjectRemoveAdd(client, sao);
        }
 
-       has_auth = m_script->getAuth(playername, &checkpwd, NULL);
-
-       if (!has_auth) {
-               actionstream << "Server: " << playername << " cannot be authenticated"
-                               << " (auth handler does not work?)" << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Not allowed to login");
-               return;
-       }
-
-       if (given_password != checkpwd) {
-               actionstream << "Server: User " << playername
-                       << " at " << addr_s
-                       << " supplied wrong password (auth mechanism: legacy)."
-                       << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Wrong password");
-               return;
-       }
-
-       RemotePlayer *player =
-                       static_cast<RemotePlayer*>(m_env->getPlayer(playername));
-
-       if (player && player->peer_id != 0) {
-               actionstream << "Server: " << playername << ": Failed to emerge player"
-                               << " (player allocated to an another client)" << std::endl;
-               DenyAccess_Legacy(pkt->getPeerId(), L"Another client is connected with this "
-                               L"name. If your client closed unexpectedly, try again in "
-                               L"a minute.");
-       }
-
-       m_clients.setPlayerName(pkt->getPeerId(), playername);
-
-       /*
-               Answer with a TOCLIENT_INIT
-       */
-
-       NetworkPacket resp_pkt(TOCLIENT_INIT_LEGACY, 1 + 6 + 8 + 4,
-                       pkt->getPeerId());
-
-       resp_pkt << (u8) deployed << (v3s16) floatToInt(v3f(0,0,0), BS)
-                       << (u64) m_env->getServerMap().getSeed()
-                       << g_settings->getFloat("dedicated_server_step");
-
-       Send(&resp_pkt);
-       m_clients.event(pkt->getPeerId(), CSE_InitLegacy);
-}
-
-void Server::handleCommand_Init2(NetworkPacket* pkt)
-{
-       verbosestream << "Server: Got TOSERVER_INIT2 from "
-                       << pkt->getPeerId() << std::endl;
-
-       m_clients.event(pkt->getPeerId(), CSE_GotInit2);
-       u16 protocol_version = m_clients.getProtocolVersion(pkt->getPeerId());
-
-
-       /*
-               Send some initialization data
-       */
-
-       infostream << "Server: Sending content to "
-                       << getPlayerName(pkt->getPeerId()) << std::endl;
+       // Send detached inventories
+       sendDetachedInventories(peer_id, false);
 
        // Send player movement settings
-       SendMovement(pkt->getPeerId());
-
-       // Send item definitions
-       SendItemDef(pkt->getPeerId(), m_itemdef, protocol_version);
-
-       // Send node definitions
-       SendNodeDef(pkt->getPeerId(), m_nodedef, protocol_version);
-
-       m_clients.event(pkt->getPeerId(), CSE_SetDefinitionsSent);
-
-       // Send media announcement
-       sendMediaAnnouncement(pkt->getPeerId());
-
-       // Send detached inventories
-       sendDetachedInventories(pkt->getPeerId());
+       SendMovement(peer_id);
 
        // Send time of day
        u16 time = m_env->getTimeOfDay();
        float time_speed = g_settings->getFloat("time_speed");
-       SendTimeOfDay(pkt->getPeerId(), time, time_speed);
+       SendTimeOfDay(peer_id, time, time_speed);
 
-       SendCSMFlavourLimits(pkt->getPeerId());
+       SendCSMRestrictionFlags(peer_id);
 
        // Warnings about protocol version can be issued here
-       if (getClient(pkt->getPeerId())->net_proto_version < LATEST_PROTOCOL_VERSION) {
-               SendChatMessage(pkt->getPeerId(), ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
-                               L"# Server: WARNING: YOUR CLIENT'S VERSION MAY NOT BE FULLY COMPATIBLE "
-                               L"WITH THIS SERVER!"));
-
+       if (client->net_proto_version < LATEST_PROTOCOL_VERSION) {
+               SendChatMessage(peer_id, ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
+                       L"# Server: WARNING: YOUR CLIENT'S VERSION MAY NOT BE FULLY COMPATIBLE "
+                       L"WITH THIS SERVER!"));
        }
 }
 
@@ -660,8 +348,9 @@ void Server::handleCommand_RequestMedia(NetworkPacket* pkt)
 
        *pkt >> numfiles;
 
-       infostream << "Sending " << numfiles << " files to "
-                       << getPlayerName(pkt->getPeerId()) << std::endl;
+       session_t peer_id = pkt->getPeerId();
+       infostream << "Sending " << numfiles << " files to " <<
+               getPlayerName(peer_id) << std::endl;
        verbosestream << "TOSERVER_REQUEST_MEDIA: " << std::endl;
 
        for (u16 i = 0; i < numfiles; i++) {
@@ -674,29 +363,27 @@ void Server::handleCommand_RequestMedia(NetworkPacket* pkt)
                                << name << std::endl;
        }
 
-       sendRequestedMedia(pkt->getPeerId(), tosend);
+       sendRequestedMedia(peer_id, tosend);
 }
 
 void Server::handleCommand_ClientReady(NetworkPacket* pkt)
 {
-       u16 peer_id = pkt->getPeerId();
+       session_t peer_id = pkt->getPeerId();
 
        PlayerSAO* playersao = StageTwoClientInit(peer_id);
 
        if (playersao == NULL) {
-               actionstream
-                       << "TOSERVER_CLIENT_READY stage 2 client init failed for peer_id: "
-                       << peer_id << std::endl;
-               m_con.DisconnectPeer(peer_id);
+               errorstream << "TOSERVER_CLIENT_READY stage 2 client init failed "
+                       "peer_id=" << peer_id << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
 
        if (pkt->getSize() < 8) {
-               errorstream
-                       << "TOSERVER_CLIENT_READY client sent inconsistent data, disconnecting peer_id: "
-                       << peer_id << std::endl;
-               m_con.DisconnectPeer(peer_id);
+               errorstream << "TOSERVER_CLIENT_READY client sent inconsistent data, "
+                       "disconnecting peer_id: " << peer_id << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
@@ -704,9 +391,11 @@ void Server::handleCommand_ClientReady(NetworkPacket* pkt)
        std::string full_ver;
        *pkt >> major_ver >> minor_ver >> patch_ver >> reserved >> full_ver;
 
-       m_clients.setClientVersion(
-                       peer_id, major_ver, minor_ver, patch_ver,
-                       full_ver);
+       m_clients.setClientVersion(peer_id, major_ver, minor_ver, patch_ver,
+               full_ver);
+
+       if (pkt->getRemainingBytes() >= 2)
+               *pkt >> playersao->getPlayer()->formspec_version;
 
        const std::vector<std::string> &players = m_clients.getPlayerNames();
        NetworkPacket list_pkt(TOCLIENT_UPDATE_PLAYER_LIST, 0, peer_id);
@@ -720,15 +409,15 @@ void Server::handleCommand_ClientReady(NetworkPacket* pkt)
        // (u16) 1 + std::string represents a pseudo vector serialization representation
        notice_pkt << (u8) PLAYER_LIST_ADD << (u16) 1 << std::string(playersao->getPlayer()->getName());
        m_clients.sendToAll(&notice_pkt);
-
        m_clients.event(peer_id, CSE_SetClientReady);
-       m_script->on_joinplayer(playersao);
+
+       s64 last_login;
+       m_script->getAuth(playersao->getPlayer()->getName(), nullptr, nullptr, &last_login);
+       m_script->on_joinplayer(playersao, last_login);
+
        // Send shutdown timer if shutdown has been scheduled
-       if (m_shutdown_timer > 0.0f) {
-               std::wstringstream ws;
-               ws << L"*** Server shutting down in "
-                               << duration_to_string(myround(m_shutdown_timer)).c_str() << ".";
-               SendChatMessage(pkt->getPeerId(), ws.str());
+       if (m_shutdown_state.isTimerRunning()) {
+               SendChatMessage(peer_id, m_shutdown_state.getShutdownTimerMessage());
        }
 }
 
@@ -765,7 +454,7 @@ void Server::handleCommand_GotBlocks(NetworkPacket* pkt)
 void Server::process_PlayerPos(RemotePlayer *player, PlayerSAO *playersao,
        NetworkPacket *pkt)
 {
-       if (pkt->getRemainingBytes() < 12 + 12 + 4 + 4)
+       if (pkt->getRemainingBytes() < 12 + 12 + 4 + 4 + 4 + 1 + 1)
                return;
 
        v3s32 ps, ss;
@@ -777,33 +466,29 @@ void Server::process_PlayerPos(RemotePlayer *player, PlayerSAO *playersao,
        *pkt >> f32pitch;
        *pkt >> f32yaw;
 
-       f32 pitch = (f32)f32pitch / 100.0;
-       f32 yaw = (f32)f32yaw / 100.0;
+       f32 pitch = (f32)f32pitch / 100.0f;
+       f32 yaw = (f32)f32yaw / 100.0f;
        u32 keyPressed = 0;
 
        // default behavior (in case an old client doesn't send these)
        f32 fov = 0;
        u8 wanted_range = 0;
 
-       if (pkt->getRemainingBytes() >= 4)
-               *pkt >> keyPressed;
-       if (pkt->getRemainingBytes() >= 1) {
-               *pkt >> f32fov;
-               fov = (f32)f32fov / 80.0;
-       }
-       if (pkt->getRemainingBytes() >= 1)
-               *pkt >> wanted_range;
+       *pkt >> keyPressed;
+       *pkt >> f32fov;
+       fov = (f32)f32fov / 80.0f;
+       *pkt >> wanted_range;
 
-       v3f position((f32)ps.X / 100.0, (f32)ps.Y / 100.0, (f32)ps.Z / 100.0);
-       v3f speed((f32)ss.X / 100.0, (f32)ss.Y / 100.0, (f32)ss.Z / 100.0);
+       v3f position((f32)ps.X / 100.0f, (f32)ps.Y / 100.0f, (f32)ps.Z / 100.0f);
+       v3f speed((f32)ss.X / 100.0f, (f32)ss.Y / 100.0f, (f32)ss.Z / 100.0f);
 
        pitch = modulo360f(pitch);
        yaw = wrapDegrees_0_360(yaw);
 
        playersao->setBasePosition(position);
        player->setSpeed(speed);
-       playersao->setPitch(pitch);
-       playersao->setYaw(yaw);
+       playersao->setLookPitch(pitch);
+       playersao->setPlayerYaw(yaw);
        playersao->setFov(fov);
        playersao->setWantedRange(wanted_range);
        player->keyPressed = keyPressed;
@@ -816,6 +501,7 @@ void Server::process_PlayerPos(RemotePlayer *player, PlayerSAO *playersao,
        player->control.sneak = (keyPressed & 64);
        player->control.LMB = (keyPressed & 128);
        player->control.RMB = (keyPressed & 256);
+       player->control.zoom = (keyPressed & 512);
 
        if (playersao->checkMovementCheat()) {
                // Call callbacks
@@ -826,21 +512,22 @@ void Server::process_PlayerPos(RemotePlayer *player, PlayerSAO *playersao,
 
 void Server::handleCommand_PlayerPos(NetworkPacket* pkt)
 {
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player object for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player object for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
@@ -886,22 +573,23 @@ void Server::handleCommand_DeletedBlocks(NetworkPacket* pkt)
 
 void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
 {
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
 
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player object for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player object for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
@@ -937,8 +625,9 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
                ma->from_inv.applyCurrentPlayer(player->getName());
                ma->to_inv.applyCurrentPlayer(player->getName());
 
-               setInventoryModified(ma->from_inv, false);
-               setInventoryModified(ma->to_inv, false);
+               m_inventory_mgr->setInventoryModified(ma->from_inv);
+               if (ma->from_inv != ma->to_inv)
+                       m_inventory_mgr->setInventoryModified(ma->to_inv);
 
                bool from_inv_is_current_player =
                        (ma->from_inv.type == InventoryLocation::PLAYER) &&
@@ -948,6 +637,18 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
                        (ma->to_inv.type == InventoryLocation::PLAYER) &&
                        (ma->to_inv.name == player->getName());
 
+               InventoryLocation *remote = from_inv_is_current_player ?
+                       &ma->to_inv : &ma->from_inv;
+
+               // Check for out-of-range interaction
+               if (remote->type == InventoryLocation::NODEMETA) {
+                       v3f node_pos   = intToFloat(remote->p, BS);
+                       v3f player_pos = player->getPlayerSAO()->getEyePosition();
+                       f32 d = player_pos.getDistanceFrom(node_pos);
+                       if (!checkInteractDistance(player, d, "inventory"))
+                               return;
+               }
+
                /*
                        Disable moving items out of craftpreview
                */
@@ -991,7 +692,7 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
 
                da->from_inv.applyCurrentPlayer(player->getName());
 
-               setInventoryModified(da->from_inv, false);
+               m_inventory_mgr->setInventoryModified(da->from_inv);
 
                /*
                        Disable dropping items out of craftpreview
@@ -1027,7 +728,7 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
 
                ca->craft_inv.applyCurrentPlayer(player->getName());
 
-               setInventoryModified(ca->craft_inv, false);
+               m_inventory_mgr->setInventoryModified(ca->craft_inv);
 
                //bool craft_inv_is_current_player =
                //      (ca->craft_inv.type == InventoryLocation::PLAYER) &&
@@ -1043,11 +744,9 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
        }
 
        // Do the action
-       a->apply(this, playersao, this);
+       a->apply(m_inventory_mgr.get(), playersao, this);
        // Eat the action
        delete a;
-
-       SendInventory(playersao);
 }
 
 void Server::handleCommand_ChatMessage(NetworkPacket* pkt)
@@ -1068,12 +767,13 @@ void Server::handleCommand_ChatMessage(NetworkPacket* pkt)
                message += (wchar_t)tmp_wchar;
        }
 
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
@@ -1084,37 +784,38 @@ void Server::handleCommand_ChatMessage(NetworkPacket* pkt)
        std::wstring answer_to_sender = handleChat(name, wname, message, true, player);
        if (!answer_to_sender.empty()) {
                // Send the answer to sender
-               SendChatMessage(pkt->getPeerId(), ChatMessage(CHATMESSAGE_TYPE_NORMAL,
-                               answer_to_sender, wname));
+               SendChatMessage(peer_id, ChatMessage(CHATMESSAGE_TYPE_NORMAL,
+                       answer_to_sender, wname));
        }
 }
 
 void Server::handleCommand_Damage(NetworkPacket* pkt)
 {
-       u8 damage;
+       u16 damage;
 
        *pkt >> damage;
 
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
 
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player object for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player object for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
-       if (g_settings->getBool("enable_damage")) {
+       if (!playersao->isImmortal()) {
                if (playersao->isDead()) {
                        verbosestream << "Server::ProcessData(): Info: "
                                "Ignoring damage as player " << player->getName()
@@ -1126,86 +827,9 @@ void Server::handleCommand_Damage(NetworkPacket* pkt)
                                << (int)damage << " hp at " << PP(playersao->getBasePosition() / BS)
                                << std::endl;
 
-               playersao->setHP(playersao->getHP() - damage);
-               SendPlayerHPOrDie(playersao);
-       }
-}
-
-void Server::handleCommand_Password(NetworkPacket* pkt)
-{
-       if (pkt->getSize() != PASSWORD_SIZE * 2)
-               return;
-
-       std::string oldpwd;
-       std::string newpwd;
-
-       // Deny for clients using the new protocol
-       RemoteClient* client = getClient(pkt->getPeerId(), CS_Created);
-       if (client->net_proto_version >= 25) {
-               infostream << "Server::handleCommand_Password(): Denying change: "
-                       << " Client protocol version for peer_id=" << pkt->getPeerId()
-                       << " too new!" << std::endl;
-               return;
-       }
-
-       for (u16 i = 0; i < PASSWORD_SIZE - 1; i++) {
-               char c = pkt->getChar(i);
-               if (c == 0)
-                       break;
-               oldpwd += c;
-       }
-
-       for (u16 i = 0; i < PASSWORD_SIZE - 1; i++) {
-               char c = pkt->getChar(PASSWORD_SIZE + i);
-               if (c == 0)
-                       break;
-               newpwd += c;
-       }
-
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
-       if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
-               return;
-       }
-
-       if (!base64_is_valid(newpwd)) {
-               infostream<<"Server: " << player->getName() <<
-                               " supplied invalid password hash" << std::endl;
-               // Wrong old password supplied!!
-               SendChatMessage(pkt->getPeerId(), ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
-                               L"Invalid new password hash supplied. Password NOT changed."));
-               return;
-       }
-
-       infostream << "Server: Client requests a password change from "
-                       << "'" << oldpwd << "' to '" << newpwd << "'" << std::endl;
-
-       std::string playername = player->getName();
-
-       std::string checkpwd;
-       m_script->getAuth(playername, &checkpwd, NULL);
-
-       if (oldpwd != checkpwd) {
-               infostream << "Server: invalid old password" << std::endl;
-               // Wrong old password supplied!!
-               SendChatMessage(pkt->getPeerId(), ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
-                               L"Invalid old password supplied. Password NOT changed."));
-               return;
-       }
-
-       bool success = m_script->setPassword(playername, newpwd);
-       if (success) {
-               actionstream << player->getName() << " changes password" << std::endl;
-               SendChatMessage(pkt->getPeerId(), ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
-                               L"Password change successful."));
-       } else {
-               actionstream << player->getName() << " tries to change password but "
-                               << "it fails" << std::endl;
-               SendChatMessage(pkt->getPeerId(), ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
-                               L"Password change failed or unavailable."));
+               PlayerHPChangeReason reason(PlayerHPChangeReason::FALL);
+               playersao->setHP((s32)playersao->getHP() - (s32)damage, reason);
+               SendPlayerHPOrDie(playersao, reason);
        }
 }
 
@@ -1214,22 +838,23 @@ void Server::handleCommand_PlayerItem(NetworkPacket* pkt)
        if (pkt->getSize() < 2)
                return;
 
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
 
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player object for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player object for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
@@ -1237,17 +862,18 @@ void Server::handleCommand_PlayerItem(NetworkPacket* pkt)
 
        *pkt >> item;
 
-       playersao->setWieldIndex(item);
+       playersao->getPlayer()->setWieldIndex(item);
 }
 
 void Server::handleCommand_Respawn(NetworkPacket* pkt)
 {
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
@@ -1257,7 +883,7 @@ void Server::handleCommand_Respawn(NetworkPacket* pkt)
        if (!playersao->isDead())
                return;
 
-       RespawnPlayer(pkt->getPeerId());
+       RespawnPlayer(peer_id);
 
        actionstream << player->getName() << " respawns at "
                        << PP(playersao->getBasePosition() / BS) << std::endl;
@@ -1266,7 +892,29 @@ void Server::handleCommand_Respawn(NetworkPacket* pkt)
        // the previous addition has been successfully removed
 }
 
-void Server::handleCommand_Interact(NetworkPacket* pkt)
+bool Server::checkInteractDistance(RemotePlayer *player, const f32 d, const std::string &what)
+{
+       ItemStack selected_item, hand_item;
+       player->getWieldedItem(&selected_item, &hand_item);
+       f32 max_d = BS * getToolRange(selected_item.getDefinition(m_itemdef),
+                       hand_item.getDefinition(m_itemdef));
+
+       // Cube diagonal * 1.5 for maximal supported node extents:
+       // sqrt(3) * 1.5 â‰… 2.6
+       if (d > max_d + 2.6f * BS) {
+               actionstream << "Player " << player->getName()
+                               << " tried to access " << what
+                               << " from too far: "
+                               << "d=" << d << ", max_d=" << max_d
+                               << "; ignoring." << std::endl;
+               // Call callbacks
+               m_script->on_cheat(player->getPlayerSAO(), "interacted_too_far");
+               return false;
+       }
+       return true;
+}
+
+void Server::handleCommand_Interact(NetworkPacket *pkt)
 {
        /*
                [0] u16 command
@@ -1275,18 +923,14 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                [5] u32 length of the next item (plen)
                [9] serialized PointedThing
                [9 + plen] player position information
-               actions:
-               0: start digging (from undersurface) or use
-               1: stop digging (all parameters ignored)
-               2: digging completed
-               3: place block or item (to abovesurface)
-               4: use item
-               5: rightclick air ("activate")
        */
-       u8 action;
+
+       InteractAction action;
        u16 item_i;
-       *pkt >> action;
+
+       *pkt >> (u8 &)action;
        *pkt >> item_i;
+
        std::istringstream tmp_is(pkt->readLongString(), std::ios::binary);
        PointedThing pointed;
        pointed.deSerialize(tmp_is);
@@ -1294,31 +938,32 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
        verbosestream << "TOSERVER_INTERACT: action=" << (int)action << ", item="
                        << item_i << ", pointed=" << pointed.dump() << std::endl;
 
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
 
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player object for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player object for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
        if (playersao->isDead()) {
-               actionstream << "Server: NoCheat: " << player->getName()
+               actionstream << "Server: " << player->getName()
                                << " tried to interact while dead; ignoring." << std::endl;
                if (pointed.type == POINTEDTHING_NODE) {
                        // Re-send block to revert change on client-side
-                       RemoteClient *client = getClient(pkt->getPeerId());
+                       RemoteClient *client = getClient(peer_id);
                        v3s16 blockpos = getNodeBlockPos(pointed.node_undersurface);
                        client->SetBlockNotSent(blockpos);
                }
@@ -1332,7 +977,7 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
        v3f player_pos = playersao->getLastGoodPosition();
 
        // Update wielded item
-       playersao->setWieldIndex(item_i);
+       playersao->getPlayer()->setWieldIndex(item_i);
 
        // Get pointed to node (undefined if not POINTEDTYPE_NODE)
        v3s16 p_under = pointed.node_undersurface;
@@ -1365,18 +1010,18 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                Make sure the player is allowed to do it
        */
        if (!checkPriv(player->getName(), "interact")) {
-               actionstream<<player->getName()<<" attempted to interact with "
-                               <<pointed.dump()<<" without 'interact' privilege"
-                               <<std::endl;
+               actionstream << player->getName() << " attempted to interact with " <<
+                               pointed.dump() << " without 'interact' privilege" << std::endl;
+
                // Re-send block to revert change on client-side
-               RemoteClient *client = getClient(pkt->getPeerId());
+               RemoteClient *client = getClient(peer_id);
                // Digging completed -> under
-               if (action == 2) {
+               if (action == INTERACT_DIGGING_COMPLETED) {
                        v3s16 blockpos = getNodeBlockPos(floatToInt(pointed_pos_under, BS));
                        client->SetBlockNotSent(blockpos);
                }
                // Placement -> above
-               if (action == 3) {
+               else if (action == INTERACT_PLACE) {
                        v3s16 blockpos = getNodeBlockPos(floatToInt(pointed_pos_above, BS));
                        client->SetBlockNotSent(blockpos);
                }
@@ -1390,34 +1035,16 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
        static thread_local const bool enable_anticheat =
                        !g_settings->getBool("disable_anticheat");
 
-       if ((action == 0 || action == 2 || action == 3 || action == 4) &&
-                       (enable_anticheat && !isSingleplayer())) {
-               float d = player_pos.getDistanceFrom(pointed_pos_under);
-               const ItemDefinition &playeritem_def =
-                       playersao->getWieldedItem().getDefinition(m_itemdef);
-               float max_d = BS * playeritem_def.range;
-               InventoryList *hlist = playersao->getInventory()->getList("hand");
-               const ItemDefinition &hand_def =
-                       hlist ? (hlist->getItem(0).getDefinition(m_itemdef)) : (m_itemdef->get(""));
-               float max_d_hand = BS * hand_def.range;
-               if (max_d < 0 && max_d_hand >= 0)
-                       max_d = max_d_hand;
-               else if (max_d < 0)
-                       max_d = BS * 4.0;
-               // cube diagonal: sqrt(3) = 1.73
-               if (d > max_d * 1.73) {
-                       actionstream << "Player " << player->getName()
-                                       << " tried to access " << pointed.dump()
-                                       << " from too far: "
-                                       << "d=" << d <<", max_d=" << max_d
-                                       << ". ignoring." << std::endl;
+       if ((action == INTERACT_START_DIGGING || action == INTERACT_DIGGING_COMPLETED ||
+                       action == INTERACT_PLACE || action == INTERACT_USE) &&
+                       enable_anticheat && !isSingleplayer()) {
+               float d = playersao->getEyePosition().getDistanceFrom(pointed_pos_under);
+
+               if (!checkInteractDistance(player, d, pointed.dump())) {
                        // Re-send block to revert change on client-side
-                       RemoteClient *client = getClient(pkt->getPeerId());
+                       RemoteClient *client = getClient(peer_id);
                        v3s16 blockpos = getNodeBlockPos(floatToInt(pointed_pos_under, BS));
                        client->SetBlockNotSent(blockpos);
-                       // Call callbacks
-                       m_script->on_cheat(playersao, "interacted_too_far");
-                       // Do nothing else
                        return;
                }
        }
@@ -1431,18 +1058,17 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
        /*
                0: start digging or punch object
        */
-       if (action == 0) {
+       if (action == INTERACT_START_DIGGING) {
                if (pointed.type == POINTEDTHING_NODE) {
                        MapNode n(CONTENT_IGNORE);
                        bool pos_ok;
 
-                       n = m_env->getMap().getNodeNoEx(p_under, &pos_ok);
+                       n = m_env->getMap().getNode(p_under, &pos_ok);
                        if (!pos_ok) {
-                               infostream << "Server: Not punching: Node not found."
-                                               << " Adding block to emerge queue."
-                                               << std::endl;
-                               m_emerge->enqueueBlockEmerge(pkt->getPeerId(),
-                                       getNodeBlockPos(p_above), false);
+                               infostream << "Server: Not punching: Node not found. "
+                                       "Adding block to emerge queue." << std::endl;
+                               m_emerge->enqueueBlockEmerge(peer_id, getNodeBlockPos(p_above),
+                                       false);
                        }
 
                        if (n.getContent() != CONTENT_IGNORE)
@@ -1452,62 +1078,66 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                        playersao->noCheatDigStart(p_under);
                }
                else if (pointed.type == POINTEDTHING_OBJECT) {
-                       // Skip if object has been removed
-                       if (pointed_object->m_removed)
+                       // Skip if object can't be interacted with anymore
+                       if (pointed_object->isGone())
                                return;
 
-                       actionstream<<player->getName()<<" punches object "
-                                       <<pointed.object_id<<": "
-                                       <<pointed_object->getDescription()<<std::endl;
-
-                       ItemStack punchitem = playersao->getWieldedItemOrHand();
+                       ItemStack selected_item, hand_item;
+                       ItemStack tool_item = playersao->getWieldedItem(&selected_item, &hand_item);
                        ToolCapabilities toolcap =
-                                       punchitem.getToolCapabilities(m_itemdef);
+                                       tool_item.getToolCapabilities(m_itemdef);
                        v3f dir = (pointed_object->getBasePosition() -
                                        (playersao->getBasePosition() + playersao->getEyeOffset())
                                                ).normalize();
                        float time_from_last_punch =
                                playersao->resetTimeFromLastPunch();
 
-                       s16 src_original_hp = pointed_object->getHP();
-                       s16 dst_origin_hp = playersao->getHP();
+                       u16 src_original_hp = pointed_object->getHP();
+                       u16 dst_origin_hp = playersao->getHP();
 
-                       pointed_object->punch(dir, &toolcap, playersao,
+                       u16 wear = pointed_object->punch(dir, &toolcap, playersao,
                                        time_from_last_punch);
 
+                       // Callback may have changed item, so get it again
+                       playersao->getWieldedItem(&selected_item);
+                       bool changed = selected_item.addWear(wear, m_itemdef);
+                       if (changed)
+                               playersao->setWieldedItem(selected_item);
+
                        // If the object is a player and its HP changed
                        if (src_original_hp != pointed_object->getHP() &&
                                        pointed_object->getType() == ACTIVEOBJECT_TYPE_PLAYER) {
-                               SendPlayerHPOrDie((PlayerSAO *)pointed_object);
+                               SendPlayerHPOrDie((PlayerSAO *)pointed_object,
+                                               PlayerHPChangeReason(PlayerHPChangeReason::PLAYER_PUNCH, playersao));
                        }
 
                        // If the puncher is a player and its HP changed
                        if (dst_origin_hp != playersao->getHP())
-                               SendPlayerHPOrDie(playersao);
+                               SendPlayerHPOrDie(playersao,
+                                               PlayerHPChangeReason(PlayerHPChangeReason::PLAYER_PUNCH, pointed_object));
                }
 
-       } // action == 0
+       } // action == INTERACT_START_DIGGING
 
        /*
                1: stop digging
        */
-       else if (action == 1) {
-       } // action == 1
+       else if (action == INTERACT_STOP_DIGGING) {
+       } // action == INTERACT_STOP_DIGGING
 
        /*
                2: Digging completed
        */
-       else if (action == 2) {
+       else if (action == INTERACT_DIGGING_COMPLETED) {
                // Only digging of nodes
                if (pointed.type == POINTEDTHING_NODE) {
                        bool pos_ok;
-                       MapNode n = m_env->getMap().getNodeNoEx(p_under, &pos_ok);
+                       MapNode n = m_env->getMap().getNode(p_under, &pos_ok);
                        if (!pos_ok) {
-                               infostream << "Server: Not finishing digging: Node not found."
-                                               << " Adding block to emerge queue."
-                                               << std::endl;
-                               m_emerge->enqueueBlockEmerge(pkt->getPeerId(),
-                                       getNodeBlockPos(p_above), false);
+                               infostream << "Server: Not finishing digging: Node not found. "
+                                       "Adding block to emerge queue." << std::endl;
+                               m_emerge->enqueueBlockEmerge(peer_id, getNodeBlockPos(p_above),
+                                       false);
                        }
 
                        /* Cheat prevention */
@@ -1518,7 +1148,7 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                                playersao->noCheatDigEnd();
                                // If player didn't start digging this, ignore dig
                                if (nocheat_p != p_under) {
-                                       infostream << "Server: NoCheat: " << player->getName()
+                                       infostream << "Server: " << player->getName()
                                                        << " started digging "
                                                        << PP(nocheat_p) << " and completed digging "
                                                        << PP(p_under) << "; not digging." << std::endl;
@@ -1526,27 +1156,25 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                                        // Call callbacks
                                        m_script->on_cheat(playersao, "finished_unknown_dig");
                                }
+
                                // Get player's wielded item
-                               ItemStack playeritem = playersao->getWieldedItemOrHand();
-                               ToolCapabilities playeritem_toolcap =
-                                               playeritem.getToolCapabilities(m_itemdef);
+                               // See also: Game::handleDigging
+                               ItemStack selected_item, hand_item;
+                               playersao->getPlayer()->getWieldedItem(&selected_item, &hand_item);
+
                                // Get diggability and expected digging time
                                DigParams params = getDigParams(m_nodedef->get(n).groups,
-                                               &playeritem_toolcap);
+                                               &selected_item.getToolCapabilities(m_itemdef));
                                // If can't dig, try hand
                                if (!params.diggable) {
-                                       InventoryList *hlist = playersao->getInventory()->getList("hand");
-                                       const ItemDefinition &hand =
-                                               hlist ? hlist->getItem(0).getDefinition(m_itemdef) : m_itemdef->get("");
-                                       const ToolCapabilities *tp = hand.tool_capabilities;
-                                       if (tp)
-                                               params = getDigParams(m_nodedef->get(n).groups, tp);
+                                       params = getDigParams(m_nodedef->get(n).groups,
+                                               &hand_item.getToolCapabilities(m_itemdef));
                                }
                                // If can't dig, ignore dig
                                if (!params.diggable) {
-                                       infostream << "Server: NoCheat: " << player->getName()
+                                       infostream << "Server: " << player->getName()
                                                        << " completed digging " << PP(p_under)
-                                                       << ", which is not diggable with tool. not digging."
+                                                       << ", which is not diggable with tool; not digging."
                                                        << std::endl;
                                        is_valid_dig = false;
                                        // Call callbacks
@@ -1570,7 +1198,7 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                                }
                                // Dig not possible
                                else {
-                                       infostream << "Server: NoCheat: " << player->getName()
+                                       infostream << "Server: " << player->getName()
                                                        << " completed digging " << PP(p_under)
                                                        << "too fast; not digging." << std::endl;
                                        is_valid_dig = false;
@@ -1585,9 +1213,9 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                                m_script->node_on_dig(p_under, n, playersao);
 
                        v3s16 blockpos = getNodeBlockPos(floatToInt(pointed_pos_under, BS));
-                       RemoteClient *client = getClient(pkt->getPeerId());
+                       RemoteClient *client = getClient(peer_id);
                        // Send unusual result (that is, node not being removed)
-                       if (m_env->getMap().getNodeNoEx(p_under).getContent() != CONTENT_AIR) {
+                       if (m_env->getMap().getNode(p_under).getContent() != CONTENT_AIR) {
                                // Re-send block to revert change on client-side
                                client->SetBlockNotSent(blockpos);
                        }
@@ -1595,24 +1223,25 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                                client->ResendBlockIfOnWire(blockpos);
                        }
                }
-       } // action == 2
+       } // action == INTERACT_DIGGING_COMPLETED
 
        /*
                3: place block or right-click object
        */
-       else if (action == 3) {
-               ItemStack item = playersao->getWieldedItem();
+       else if (action == INTERACT_PLACE) {
+               ItemStack selected_item;
+               playersao->getWieldedItem(&selected_item, nullptr);
 
                // Reset build time counter
                if (pointed.type == POINTEDTHING_NODE &&
-                               item.getDefinition(m_itemdef).type == ITEM_NODE)
-                       getClient(pkt->getPeerId())->m_time_from_building = 0.0;
+                               selected_item.getDefinition(m_itemdef).type == ITEM_NODE)
+                       getClient(peer_id)->m_time_from_building = 0.0;
 
                if (pointed.type == POINTEDTHING_OBJECT) {
                        // Right click object
 
-                       // Skip if object has been removed
-                       if (pointed_object->m_removed)
+                       // Skip if object can't be interacted with anymore
+                       if (pointed_object->isGone())
                                return;
 
                        actionstream << player->getName() << " right-clicks object "
@@ -1620,24 +1249,30 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                                        << pointed_object->getDescription() << std::endl;
 
                        // Do stuff
+                       if (m_script->item_OnSecondaryUse(
+                                       selected_item, playersao, pointed)) {
+                               if (playersao->setWieldedItem(selected_item)) {
+                                       SendInventory(playersao, true);
+                               }
+                       }
+
                        pointed_object->rightClick(playersao);
-               }
-               else if (m_script->item_OnPlace(
-                               item, playersao, pointed)) {
+               } else if (m_script->item_OnPlace(
+                               selected_item, playersao, pointed)) {
                        // Placement was handled in lua
 
                        // Apply returned ItemStack
-                       if (playersao->setWieldedItem(item)) {
-                               SendInventory(playersao);
+                       if (playersao->setWieldedItem(selected_item)) {
+                               SendInventory(playersao, true);
                        }
                }
 
                // If item has node placement prediction, always send the
                // blocks to make sure the client knows what exactly happened
-               RemoteClient *client = getClient(pkt->getPeerId());
+               RemoteClient *client = getClient(peer_id);
                v3s16 blockpos = getNodeBlockPos(floatToInt(pointed_pos_above, BS));
                v3s16 blockpos2 = getNodeBlockPos(floatToInt(pointed_pos_under, BS));
-               if (item.getDefinition(m_itemdef).node_placement_prediction != "") {
+               if (!selected_item.getDefinition(m_itemdef).node_placement_prediction.empty()) {
                        client->SetBlockNotSent(blockpos);
                        if (blockpos2 != blockpos) {
                                client->SetBlockNotSent(blockpos2);
@@ -1649,43 +1284,47 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                                client->ResendBlockIfOnWire(blockpos2);
                        }
                }
-       } // action == 3
+       } // action == INTERACT_PLACE
 
        /*
                4: use
        */
-       else if (action == 4) {
-               ItemStack item = playersao->getWieldedItem();
+       else if (action == INTERACT_USE) {
+               ItemStack selected_item;
+               playersao->getWieldedItem(&selected_item, nullptr);
 
-               actionstream << player->getName() << " uses " << item.name
+               actionstream << player->getName() << " uses " << selected_item.name
                                << ", pointing at " << pointed.dump() << std::endl;
 
                if (m_script->item_OnUse(
-                               item, playersao, pointed)) {
+                               selected_item, playersao, pointed)) {
                        // Apply returned ItemStack
-                       if (playersao->setWieldedItem(item)) {
-                               SendInventory(playersao);
+                       if (playersao->setWieldedItem(selected_item)) {
+                               SendInventory(playersao, true);
                        }
                }
 
-       } // action == 4
+       } // action == INTERACT_USE
 
        /*
                5: rightclick air
        */
-       else if (action == 5) {
-               ItemStack item = playersao->getWieldedItem();
+       else if (action == INTERACT_ACTIVATE) {
+               ItemStack selected_item;
+               playersao->getWieldedItem(&selected_item, nullptr);
 
                actionstream << player->getName() << " activates "
-                               << item.name << std::endl;
+                               << selected_item.name << std::endl;
+
+               pointed.type = POINTEDTHING_NOTHING; // can only ever be NOTHING
 
                if (m_script->item_OnSecondaryUse(
-                               item, playersao)) {
-                       if( playersao->setWieldedItem(item)) {
-                               SendInventory(playersao);
+                               selected_item, playersao, pointed)) {
+                       if (playersao->setWieldedItem(selected_item)) {
+                               SendInventory(playersao, true);
                        }
                }
-       }
+       } // action == INTERACT_ACTIVATE
 
 
        /*
@@ -1733,22 +1372,23 @@ void Server::handleCommand_NodeMetaFields(NetworkPacket* pkt)
                fields[fieldname] = pkt->readLongString();
        }
 
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
 
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player object for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!"  << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player object for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
@@ -1772,10 +1412,10 @@ void Server::handleCommand_NodeMetaFields(NetworkPacket* pkt)
 
 void Server::handleCommand_InventoryFields(NetworkPacket* pkt)
 {
-       std::string formname;
+       std::string client_formspec_name;
        u16 num;
 
-       *pkt >> formname >> num;
+       *pkt >> client_formspec_name >> num;
 
        StringMap fields;
        for (u16 k = 0; k < num; k++) {
@@ -1784,31 +1424,60 @@ void Server::handleCommand_InventoryFields(NetworkPacket* pkt)
                fields[fieldname] = pkt->readLongString();
        }
 
-       RemotePlayer *player = m_env->getPlayer(pkt->getPeerId());
+       session_t peer_id = pkt->getPeerId();
+       RemotePlayer *player = m_env->getPlayer(peer_id);
 
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
                return;
        }
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Canceling: "
-                               "No player object for peer_id=" << pkt->getPeerId()
-                               << " disconnecting peer!" << std::endl;
-               m_con.DisconnectPeer(pkt->getPeerId());
+               errorstream <<
+                       "Server::ProcessData(): Canceling: No player object for peer_id=" <<
+                       peer_id << " disconnecting peer!" << std::endl;
+               DisconnectPeer(peer_id);
+               return;
+       }
+
+       if (client_formspec_name.empty()) { // pass through inventory submits
+               m_script->on_playerReceiveFields(playersao, client_formspec_name, fields);
                return;
        }
 
-       m_script->on_playerReceiveFields(playersao, formname, fields);
+       // verify that we displayed the formspec to the user
+       const auto peer_state_iterator = m_formspec_state_data.find(peer_id);
+       if (peer_state_iterator != m_formspec_state_data.end()) {
+               const std::string &server_formspec_name = peer_state_iterator->second;
+               if (client_formspec_name == server_formspec_name) {
+                       auto it = fields.find("quit");
+                       if (it != fields.end() && it->second == "true")
+                               m_formspec_state_data.erase(peer_state_iterator);
+
+                       m_script->on_playerReceiveFields(playersao, client_formspec_name, fields);
+                       return;
+               }
+               actionstream << "'" << player->getName()
+                       << "' submitted formspec ('" << client_formspec_name
+                       << "') but the name of the formspec doesn't match the"
+                       " expected name ('" << server_formspec_name << "')";
+
+       } else {
+               actionstream << "'" << player->getName()
+                       << "' submitted formspec ('" << client_formspec_name
+                       << "') but server hasn't sent formspec to client";
+       }
+       actionstream << ", possible exploitation attempt" << std::endl;
 }
 
 void Server::handleCommand_FirstSrp(NetworkPacket* pkt)
 {
-       RemoteClient* client = getClient(pkt->getPeerId(), CS_Invalid);
+       session_t peer_id = pkt->getPeerId();
+       RemoteClient *client = getClient(peer_id, CS_Invalid);
        ClientState cstate = client->getState();
 
        std::string playername = client->getName();
@@ -1816,7 +1485,7 @@ void Server::handleCommand_FirstSrp(NetworkPacket* pkt)
        std::string salt;
        std::string verification_key;
 
-       std::string addr_s = getPeerAddress(pkt->getPeerId()).serializeString();
+       std::string addr_s = getPeerAddress(peer_id).serializeString();
        u8 is_empty;
 
        *pkt >> salt >> verification_key >> is_empty;
@@ -1830,7 +1499,7 @@ void Server::handleCommand_FirstSrp(NetworkPacket* pkt)
                        actionstream << "Server: Client from " << addr_s
                                        << " tried to set password without being "
                                        << "authenticated, or the username being new." << std::endl;
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+                       DenyAccess(peer_id, SERVER_ACCESSDENIED_UNEXPECTED_DATA);
                        return;
                }
 
@@ -1839,7 +1508,7 @@ void Server::handleCommand_FirstSrp(NetworkPacket* pkt)
                                is_empty == 1) {
                        actionstream << "Server: " << playername
                                        << " supplied empty password from " << addr_s << std::endl;
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_EMPTY_PASSWORD);
+                       DenyAccess(peer_id, SERVER_ACCESSDENIED_EMPTY_PASSWORD);
                        return;
                }
 
@@ -1847,8 +1516,9 @@ void Server::handleCommand_FirstSrp(NetworkPacket* pkt)
 
                initial_ver_key = encode_srp_verifier(verification_key, salt);
                m_script->createAuth(playername, initial_ver_key);
+               m_script->on_authplayer(playername, addr_s, true);
 
-               acceptAuth(pkt->getPeerId(), false);
+               acceptAuth(peer_id, false);
        } else {
                if (cstate < CS_SudoMode) {
                        infostream << "Server::ProcessData(): Ignoring TOSERVER_FIRST_SRP from "
@@ -1856,49 +1526,49 @@ void Server::handleCommand_FirstSrp(NetworkPacket* pkt)
                                        << std::endl;
                        return;
                }
-               m_clients.event(pkt->getPeerId(), CSE_SudoLeave);
+               m_clients.event(peer_id, CSE_SudoLeave);
                std::string pw_db_field = encode_srp_verifier(verification_key, salt);
                bool success = m_script->setPassword(playername, pw_db_field);
                if (success) {
                        actionstream << playername << " changes password" << std::endl;
-                       SendChatMessage(pkt->getPeerId(), ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
-                                       L"Password change successful."));
+                       SendChatMessage(peer_id, ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
+                               L"Password change successful."));
                } else {
-                       actionstream << playername << " tries to change password but "
-                               << "it fails" << std::endl;
-                       SendChatMessage(pkt->getPeerId(), ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
-                                       L"Password change failed or unavailable."));
+                       actionstream << playername <<
+                               " tries to change password but it fails" << std::endl;
+                       SendChatMessage(peer_id, ChatMessage(CHATMESSAGE_TYPE_SYSTEM,
+                               L"Password change failed or unavailable."));
                }
        }
 }
 
 void Server::handleCommand_SrpBytesA(NetworkPacket* pkt)
 {
-       RemoteClient* client = getClient(pkt->getPeerId(), CS_Invalid);
+       session_t peer_id = pkt->getPeerId();
+       RemoteClient *client = getClient(peer_id, CS_Invalid);
        ClientState cstate = client->getState();
 
        bool wantSudo = (cstate == CS_Active);
 
        if (!((cstate == CS_HelloSent) || (cstate == CS_Active))) {
-               actionstream << "Server: got SRP _A packet in wrong state "
-                       << cstate << " from "
-                       << getPeerAddress(pkt->getPeerId()).serializeString()
-                       << ". Ignoring." << std::endl;
+               actionstream << "Server: got SRP _A packet in wrong state " << cstate <<
+                       " from " << getPeerAddress(peer_id).serializeString() <<
+                       ". Ignoring." << std::endl;
                return;
        }
 
        if (client->chosen_mech != AUTH_MECHANISM_NONE) {
-               actionstream << "Server: got SRP _A packet, while auth"
-                       << "is already going on with mech " << client->chosen_mech
-                       << " from " << getPeerAddress(pkt->getPeerId()).serializeString()
-                       << " (wantSudo=" << wantSudo << "). Ignoring." << std::endl;
+               actionstream << "Server: got SRP _A packet, while auth is already "
+                       "going on with mech " << client->chosen_mech << " from " <<
+                       getPeerAddress(peer_id).serializeString() <<
+                       " (wantSudo=" << wantSudo << "). Ignoring." << std::endl;
                if (wantSudo) {
-                       DenySudoAccess(pkt->getPeerId());
-                       return;
-               } else {
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+                       DenySudoAccess(peer_id);
                        return;
                }
+
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+               return;
        }
 
        std::string bytes_A;
@@ -1914,19 +1584,19 @@ void Server::handleCommand_SrpBytesA(NetworkPacket* pkt)
 
        if (wantSudo) {
                if (!client->isSudoMechAllowed(chosen)) {
-                       actionstream << "Server: Player \"" << client->getName()
-                               << "\" at " << getPeerAddress(pkt->getPeerId()).serializeString()
-                               << " tried to change password using unallowed mech "
-                               << chosen << "." << std::endl;
-                       DenySudoAccess(pkt->getPeerId());
+                       actionstream << "Server: Player \"" << client->getName() <<
+                               "\" at " << getPeerAddress(peer_id).serializeString() <<
+                               " tried to change password using unallowed mech " << chosen <<
+                               "." << std::endl;
+                       DenySudoAccess(peer_id);
                        return;
                }
        } else {
                if (!client->isMechAllowed(chosen)) {
-                       actionstream << "Server: Client tried to authenticate from "
-                               << getPeerAddress(pkt->getPeerId()).serializeString()
-                               << " using unallowed mech " << chosen << "." << std::endl;
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+                       actionstream << "Server: Client tried to authenticate from " <<
+                               getPeerAddress(peer_id).serializeString() <<
+                               " using unallowed mech " << chosen << "." << std::endl;
+                       DenyAccess(peer_id, SERVER_ACCESSDENIED_UNEXPECTED_DATA);
                        return;
                }
        }
@@ -1942,10 +1612,10 @@ void Server::handleCommand_SrpBytesA(NetworkPacket* pkt)
                        &verifier, &salt);
        } else if (!decode_srp_verifier_and_salt(client->enc_pwd, &verifier, &salt)) {
                // Non-base64 errors should have been catched in the init handler
-               actionstream << "Server: User " << client->getName()
-                       << " tried to log in, but srp verifier field"
-                       << " was invalid (most likely invalid base64)." << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_SERVER_FAIL);
+               actionstream << "Server: User " << client->getName() <<
+                       " tried to log in, but srp verifier field was invalid (most likely "
+                       "invalid base64)." << std::endl;
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_SERVER_FAIL);
                return;
        }
 
@@ -1965,49 +1635,50 @@ void Server::handleCommand_SrpBytesA(NetworkPacket* pkt)
                        << " tried to log in, SRP-6a safety check violated in _A handler."
                        << std::endl;
                if (wantSudo) {
-                       DenySudoAccess(pkt->getPeerId());
-                       return;
-               } else {
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+                       DenySudoAccess(peer_id);
                        return;
                }
+
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+               return;
        }
 
-       NetworkPacket resp_pkt(TOCLIENT_SRP_BYTES_S_B, 0, pkt->getPeerId());
+       NetworkPacket resp_pkt(TOCLIENT_SRP_BYTES_S_B, 0, peer_id);
        resp_pkt << salt << std::string(bytes_B, len_B);
        Send(&resp_pkt);
 }
 
 void Server::handleCommand_SrpBytesM(NetworkPacket* pkt)
 {
-       RemoteClient* client = getClient(pkt->getPeerId(), CS_Invalid);
+       session_t peer_id = pkt->getPeerId();
+       RemoteClient *client = getClient(peer_id, CS_Invalid);
        ClientState cstate = client->getState();
+       std::string addr_s = getPeerAddress(pkt->getPeerId()).serializeString();
+       std::string playername = client->getName();
 
        bool wantSudo = (cstate == CS_Active);
 
-       verbosestream << "Server: Recieved TOCLIENT_SRP_BYTES_M." << std::endl;
+       verbosestream << "Server: Received TOCLIENT_SRP_BYTES_M." << std::endl;
 
        if (!((cstate == CS_HelloSent) || (cstate == CS_Active))) {
                actionstream << "Server: got SRP _M packet in wrong state "
-                       << cstate << " from "
-                       << getPeerAddress(pkt->getPeerId()).serializeString()
+                       << cstate << " from " << addr_s
                        << ". Ignoring." << std::endl;
                return;
        }
 
-       if ((client->chosen_mech != AUTH_MECHANISM_SRP)
-               && (client->chosen_mech != AUTH_MECHANISM_LEGACY_PASSWORD)) {
+       if (client->chosen_mech != AUTH_MECHANISM_SRP &&
+                       client->chosen_mech != AUTH_MECHANISM_LEGACY_PASSWORD) {
                actionstream << "Server: got SRP _M packet, while auth"
-                       << "is going on with mech " << client->chosen_mech
-                       << " from " << getPeerAddress(pkt->getPeerId()).serializeString()
-                       << " (wantSudo=" << wantSudo << "). Denying." << std::endl;
+                       << "is going on with mech " << client->chosen_mech << " from " 
+                       << addr_s << " (wantSudo=" << wantSudo << "). Denying." << std::endl;
                if (wantSudo) {
-                       DenySudoAccess(pkt->getPeerId());
-                       return;
-               } else {
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+                       DenySudoAccess(peer_id);
                        return;
                }
+
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+               return;
        }
 
        std::string bytes_M;
@@ -2015,10 +1686,9 @@ void Server::handleCommand_SrpBytesM(NetworkPacket* pkt)
 
        if (srp_verifier_get_session_key_length((SRPVerifier *) client->auth_data)
                        != bytes_M.size()) {
-               actionstream << "Server: User " << client->getName()
-                       << " at " << getPeerAddress(pkt->getPeerId()).serializeString()
+               actionstream << "Server: User " << playername << " at " << addr_s
                        << " sent bytes_M with invalid length " << bytes_M.size() << std::endl;
-               DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_UNEXPECTED_DATA);
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_UNEXPECTED_DATA);
                return;
        }
 
@@ -2029,35 +1699,116 @@ void Server::handleCommand_SrpBytesM(NetworkPacket* pkt)
 
        if (!bytes_HAMK) {
                if (wantSudo) {
-                       actionstream << "Server: User " << client->getName()
-                               << " at " << getPeerAddress(pkt->getPeerId()).serializeString()
+                       actionstream << "Server: User " << playername << " at " << addr_s
                                << " tried to change their password, but supplied wrong"
                                << " (SRP) password for authentication." << std::endl;
-                       DenySudoAccess(pkt->getPeerId());
-                       return;
-               } else {
-                       actionstream << "Server: User " << client->getName()
-                               << " at " << getPeerAddress(pkt->getPeerId()).serializeString()
-                               << " supplied wrong password (auth mechanism: SRP)."
-                               << std::endl;
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_WRONG_PASSWORD);
+                       DenySudoAccess(peer_id);
                        return;
                }
+
+               actionstream << "Server: User " << playername << " at " << addr_s
+                       << " supplied wrong password (auth mechanism: SRP)." << std::endl;
+               m_script->on_authplayer(playername, addr_s, false);
+               DenyAccess(peer_id, SERVER_ACCESSDENIED_WRONG_PASSWORD);
+               return;
        }
 
        if (client->create_player_on_auth_success) {
-               std::string playername = client->getName();
                m_script->createAuth(playername, client->enc_pwd);
 
                std::string checkpwd; // not used, but needed for passing something
                if (!m_script->getAuth(playername, &checkpwd, NULL)) {
-                       actionstream << "Server: " << playername << " cannot be authenticated"
-                               << " (auth handler does not work?)" << std::endl;
-                       DenyAccess(pkt->getPeerId(), SERVER_ACCESSDENIED_SERVER_FAIL);
+                       actionstream << "Server: " << playername <<
+                               " cannot be authenticated (auth handler does not work?)" <<
+                               std::endl;
+                       DenyAccess(peer_id, SERVER_ACCESSDENIED_SERVER_FAIL);
                        return;
                }
                client->create_player_on_auth_success = false;
        }
 
-       acceptAuth(pkt->getPeerId(), wantSudo);
+       m_script->on_authplayer(playername, addr_s, true);
+       acceptAuth(peer_id, wantSudo);
+}
+
+/*
+ * Mod channels
+ */
+
+void Server::handleCommand_ModChannelJoin(NetworkPacket *pkt)
+{
+       std::string channel_name;
+       *pkt >> channel_name;
+
+       session_t peer_id = pkt->getPeerId();
+       NetworkPacket resp_pkt(TOCLIENT_MODCHANNEL_SIGNAL,
+               1 + 2 + channel_name.size(), peer_id);
+
+       // Send signal to client to notify join succeed or not
+       if (g_settings->getBool("enable_mod_channels") &&
+                       m_modchannel_mgr->joinChannel(channel_name, peer_id)) {
+               resp_pkt << (u8) MODCHANNEL_SIGNAL_JOIN_OK;
+               infostream << "Peer " << peer_id << " joined channel " <<
+                       channel_name << std::endl;
+       }
+       else {
+               resp_pkt << (u8)MODCHANNEL_SIGNAL_JOIN_FAILURE;
+               infostream << "Peer " << peer_id << " tried to join channel " <<
+                       channel_name << ", but was already registered." << std::endl;
+       }
+       resp_pkt << channel_name;
+       Send(&resp_pkt);
+}
+
+void Server::handleCommand_ModChannelLeave(NetworkPacket *pkt)
+{
+       std::string channel_name;
+       *pkt >> channel_name;
+
+       session_t peer_id = pkt->getPeerId();
+       NetworkPacket resp_pkt(TOCLIENT_MODCHANNEL_SIGNAL,
+               1 + 2 + channel_name.size(), peer_id);
+
+       // Send signal to client to notify join succeed or not
+       if (g_settings->getBool("enable_mod_channels") &&
+                       m_modchannel_mgr->leaveChannel(channel_name, peer_id)) {
+               resp_pkt << (u8)MODCHANNEL_SIGNAL_LEAVE_OK;
+               infostream << "Peer " << peer_id << " left channel " << channel_name <<
+                       std::endl;
+       } else {
+               resp_pkt << (u8) MODCHANNEL_SIGNAL_LEAVE_FAILURE;
+               infostream << "Peer " << peer_id << " left channel " << channel_name <<
+                       ", but was not registered." << std::endl;
+       }
+       resp_pkt << channel_name;
+       Send(&resp_pkt);
+}
+
+void Server::handleCommand_ModChannelMsg(NetworkPacket *pkt)
+{
+       std::string channel_name, channel_msg;
+       *pkt >> channel_name >> channel_msg;
+
+       session_t peer_id = pkt->getPeerId();
+       verbosestream << "Mod channel message received from peer " << peer_id <<
+               " on channel " << channel_name << " message: " << channel_msg <<
+               std::endl;
+
+       // If mod channels are not enabled, discard message
+       if (!g_settings->getBool("enable_mod_channels")) {
+               return;
+       }
+
+       // If channel not registered, signal it and ignore message
+       if (!m_modchannel_mgr->channelRegistered(channel_name)) {
+               NetworkPacket resp_pkt(TOCLIENT_MODCHANNEL_SIGNAL,
+                       1 + 2 + channel_name.size(), peer_id);
+               resp_pkt << (u8)MODCHANNEL_SIGNAL_CHANNEL_NOT_REGISTERED << channel_name;
+               Send(&resp_pkt);
+               return;
+       }
+
+       // @TODO: filter, rate limit
+
+       broadcastModChannelMessage(channel_name, channel_msg, peer_id);
 }