mp911de / lettuce / 1081

92%

push

travis-ci

Add ConnectionWatchdog as last handler #335

The reuse of handlers causes multiple calls to channelActive/channelInactive/and so on. In some cases, the reconnection is faster than channel deregistration (calls to channelUnregistered) which leads to the call sequence channelRegistered (new channel), channelUnregistered (old channel). Similar for channelActive/channelInactive. This is handled inside the CommandHandler but can still lead to issues. Moving the ConnectionWatchdog to the end reduces the probability of the calls coming in the wrong order. It fixes also the channelActive/channelInactive order in fast reconnect scenarios as the channelInactive call is done before notifying the ConnectionWatchdog.

4 of 4 new or added lines in 2 files covered. (100.0%)

10018 of 10790 relevant lines covered (92.85%)

1.85 hits per line