Skip to content

Commit 07b8d48

Browse files
committed
If socket is already connected, just return true
Socket connection may have been finished elsewhere, such as by IO.select. If already connected, just return true.
1 parent 1b484dc commit 07b8d48

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

core/src/main/java/org/jruby/ext/socket/RubySocket.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -522,7 +522,9 @@ private boolean tryConnect(ThreadContext context, Channel channel, SocketAddress
522522
if (channel instanceof SocketChannel) {
523523
SocketChannel socket = (SocketChannel) channel;
524524

525-
if (socket.isConnectionPending()) {
525+
if (socket.isConnected()) {
526+
return true;
527+
} else if (socket.isConnectionPending()) {
526528
// connection initiated but not finished
527529
result = socket.finishConnect();
528530
} else {

0 commit comments

Comments
 (0)