Возник вопрос, а где закрытие ?

public void run() {
            String userName = null;
            Connection connection = null;
            try{
                ConsoleHelper.writeMessage("Новое соединение "+ socket.getRemoteSocketAddress());
                connection = new Connection(socket);
                userName = serverHandshake(connection);
                sendBroadcastMessage(new Message(MessageType.USER_ADDED,userName));
                sendListOfUsers(connection,userName);
                serverMainLoop(connection,userName);
            }
            catch (ClassNotFoundException e){
                e.printStackTrace();
                ConsoleHelper.writeMessage("Произошла ошибка при обмене данными с удаленным адресом");
            }
            catch (IOException e){
                e.printStackTrace();
                ConsoleHelper.writeMessage("Произошла ошибка при обмене данными с удаленным адресом");
            }

            if(userName != null && !userName.isEmpty()) {
                connectionMap.remove(userName);
                sendBroadcastMessage(new Message(MessageType.USER_REMOVED, userName));
            }

            ConsoleHelper.writeMessage("Соединение закрыто");
        }
    }

К сожалению, я вам не скажу, почему валидатор принял такой код. Лично я делал через try с ресурсами:

try(Connection connection = new Connection(socket)){
...
}

В этом варианте при выходе из try - все ресурсы которые были созданы будут автоматически закрыты. Либо я чего-то не понимаю, либо недочет валидатора. В общем буду следить за темой =)

Ага, спасибо ) Я попробую переделать и сдать с закрытием )