Skip to content

Commit 32c72d1

Browse files
committed
JVMCBC-1702 PipelineErrorHandler.channelInactive fires wrong event
Motivation ---------- Should fire "channel inactive" instead of "channel active". Currently, this doesn't matter since it's the last handler in the pipeline, but let's fix it anyway in case another handler is added in the future. Change-Id: I0fe87adcff4edd0734df77d86767554950d89c3e Reviewed-on: https://review.couchbase.org/c/couchbase-jvm-clients/+/235873 Tested-by: Build Bot <[email protected]> Reviewed-by: Michael Reiche <[email protected]>
1 parent 96005da commit 32c72d1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

core-io/src/main/java/com/couchbase/client/core/io/netty/PipelineErrorHandler.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ public void exceptionCaught(final ChannelHandlerContext ctx, final Throwable cau
7878
@Override
7979
public void channelInactive(ChannelHandlerContext ctx) {
8080
endpoint.notifyChannelInactive();
81-
ctx.fireChannelActive();
81+
ctx.fireChannelInactive();
8282
}
8383

8484
/**

0 commit comments

Comments
 (0)