Remove redundant obsolete connection-checking code.

This commit is contained in:
Andrew Steinborn
2018-10-15 17:06:37 -04:00
parent 8eb7ecba83
commit 8ec273050b

View File

@@ -93,7 +93,7 @@ public class BackendPlaySessionHandler implements MinecraftSessionHandler {
serverConn.getPlayer().sendLegacyForgeHandshakeResetPacket(); serverConn.getPlayer().sendLegacyForgeHandshakeResetPacket();
} }
// Always forward these messages during login. Don't pass it onto the handleGeneric below. // Always forward these messages during login.
return false; return false;
} }
@@ -126,26 +126,11 @@ public class BackendPlaySessionHandler implements MinecraftSessionHandler {
@Override @Override
public void handleGeneric(MinecraftPacket packet) { public void handleGeneric(MinecraftPacket packet) {
if (!serverConn.getPlayer().isActive()) {
// Connection was left open accidentally. Close it so as to avoid "You logged in from another location"
// errors.
serverConn.disconnect();
return;
}
// Just forward the packet on. We don't have anything to handle at this time.
serverConn.getPlayer().getConnection().write(packet); serverConn.getPlayer().getConnection().write(packet);
} }
@Override @Override
public void handleUnknown(ByteBuf buf) { public void handleUnknown(ByteBuf buf) {
if (!serverConn.getPlayer().isActive()) {
// Connection was left open accidentally. Close it so as to avoid "You logged in from another location"
// errors.
serverConn.disconnect();
return;
}
serverConn.getPlayer().getConnection().write(buf.retain()); serverConn.getPlayer().getConnection().write(buf.retain());
} }