andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Make sure Channel connected event is not fired on connect failure. See #249

Browse Source
This commit is contained in:
norman 2012-04-04 07:43:52 +02:00
parent 6995701f20
commit 33f9d55bc4
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/main/java/org/jboss/netty/channel/socket/nio/NioWorker.java
View File

@ -186,6 +186,11 @@ public class NioWorker extends AbstractNioWorker {
channel.setConnected(); channel.setConnected();
future.setSuccess(); future.setSuccess();
} }
if (server || !((NioClientSocketChannel) channel).boundManually) {
fireChannelBound(channel, localAddress);
}
fireChannelConnected(channel, remoteAddress);
} catch (IOException e) { } catch (IOException e) {
if (future != null) { if (future != null) {
future.setFailure(e); future.setFailure(e);
@ -197,10 +202,6 @@ public class NioWorker extends AbstractNioWorker {
} }
} }
if (server || !((NioClientSocketChannel) channel).boundManually) {
fireChannelBound(channel, localAddress);
}
fireChannelConnected(channel, remoteAddress);
} }
} }