Alawn/Chronicle-Map
1
0
Fork 0
mirror of https://github.com/OpenHFT/Chronicle-Map.git synced 2021-09-13 01:26:12 +00:00
Code Releases Activity

HCOLL-309 added a return and now outputting the remote connection.

Browse Source
This commit is contained in:
Rob Austin
2015-02-23 09:58:28 +00:00
parent c6ee76d49b
commit 5b2950e420
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/java/net/openhft/chronicle/map/TcpReplicator.java
+10 -3
View File
@@ -32,7 +32,6 @@ import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import java.io.Closeable;
import java.io.EOFException;
import java.io.IOException;
import java.net.*;
import java.nio.BufferUnderflowException;
@@ -648,15 +647,23 @@ final class TcpReplicator<K, V> extends AbstractChannelReplicator implements Clo
attached.serverVersion = reader.readRemoteServerVersion();
} catch (IllegalStateException e1) {
socketChannel.close();
return;
}
if (attached.serverVersion == null)
return;
if (!isValidVersionNumber(attached.serverVersion)) {
LOG.warn("Please check that you don't have a third party system incorrectly connecting to ChronicleMap, " +
"Closing the remote connection as Chronicle can not make sense of the remote version number received from the external connection, version="+attached.serverVersion+", Chronicle is expecting the version number to only contain '.',A-Z,a-z,0-9");
LOG.warn("Closing the remote connection : Please check that you don't have a third " +
"party system incorrectly " +
"connecting to ChronicleMap, remoteAddress=" + socketChannel.getRemoteAddress() +
", so closing the remote connection as Chronicle can not make sense of the " +
"remote version number received from the external connection, " +
"version=" + attached.serverVersion + ", " +
"Chronicle is expecting the version number to only contain " +
"'.','-', ,A-Z,a-z,0-9");
socketChannel.close();
return;
}
checkVersions(attached);