Full support for timeout
-
Hi,
I found that the TCP connection of Modbus4j does not fully support timeout features of Java sockets.
Only the connection timeout is controlled in current implementation.
It can be easily extended by controlling the read timeout too.For this reason one can consider changing TcpMaster.openConnection() method part:
Before:
socket = new Socket();
socket.connect(new InetSocketAddress(ipParameters.getHost(), ipParameters.getPort()), getTimeout());
transport = new StreamTransport(socket.getInputStream(), socket.getOutputStream());After:
socket = new Socket();
socket.setSoTimeout(getTimeout());
socket.connect(new InetSocketAddress(ipParameters.getHost(), ipParameters.getPort()), getTimeout());
transport = new StreamTransport(socket.getInputStream(), socket.getOutputStream());This might be useful for somebody.
-
Thanks for the suggestion. This has been added.
-
another suggertion to
Enable/disable TCP_NODELAY (disable/enable Nagle's algorithm);socket.setTcpNoDelay(getTcpNoDelay);