Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Server: Improve some log messages (#9820)
  • Loading branch information
sfan5 committed May 5, 2020
1 parent f34c62c commit d1c6cc7
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 12 deletions.
16 changes: 7 additions & 9 deletions src/network/serverpackethandler.cpp
Expand Up @@ -111,8 +111,6 @@ void Server::handleCommand_Init(NetworkPacket* pkt)

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 << " client_max=" << (int)client_max << std::endl;
DenyAccess(peer_id, SERVER_ACCESSDENIED_WRONG_VERSION);
return;
Expand Down Expand Up @@ -902,8 +900,8 @@ bool Server::checkInteractDistance(RemotePlayer *player, const f32 d, const std:
actionstream << "Player " << player->getName()
<< " tried to access " << what
<< " from too far: "
<< "d=" << d <<", max_d=" << max_d
<< ". ignoring." << std::endl;
<< "d=" << d << ", max_d=" << max_d
<< "; ignoring." << std::endl;
// Call callbacks
m_script->on_cheat(player->getPlayerSAO(), "interacted_too_far");
return false;
Expand Down Expand Up @@ -956,7 +954,7 @@ void Server::handleCommand_Interact(NetworkPacket *pkt)
}

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
Expand Down Expand Up @@ -1145,7 +1143,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;
Expand All @@ -1169,9 +1167,9 @@ void Server::handleCommand_Interact(NetworkPacket *pkt)
}
// 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
Expand All @@ -1195,7 +1193,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;
Expand Down
6 changes: 3 additions & 3 deletions src/server/player_sao.cpp
Expand Up @@ -656,9 +656,9 @@ bool PlayerSAO::checkMovementCheat()
float lag_pool_max = m_env->getMaxLagEstimate() * 2.0;
lag_pool_max = MYMAX(lag_pool_max, LAG_POOL_MIN);
if (m_time_from_last_teleport > lag_pool_max) {
actionstream << "Player " << m_player->getName()
<< " moved too fast; resetting position"
<< std::endl;
actionstream << "Server: " << m_player->getName()
<< " moved too fast: V=" << d_vert << ", H=" << d_horiz
<< "; resetting position." << std::endl;
cheated = true;
}
setBasePosition(m_last_good_position);
Expand Down

0 comments on commit d1c6cc7

Please sign in to comment.