Fixed - PingConnectionHandler should close channel on RedisLoadingExc…

…eption, RedisTryAgainException, RedisClusterDownException, RedisBusyException redisson#4440
yxc6123 · Aug 23, 2022 · 7aa2775 · 7aa2775
1 parent 22dccb7
commit 7aa2775
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/redisson/src/main/java/org/redisson/client/handler/PingConnectionHandler.java b/redisson/src/main/java/org/redisson/client/handler/PingConnectionHandler.java
@@ -87,6 +87,7 @@ private void sendPing(ChannelHandlerContext ctx) {
                         || cause instanceof RedisTryAgainException
                             || cause instanceof RedisClusterDownException
                                 || cause instanceof RedisBusyException)) {
+                    log.debug("channel: {} closed due to PING response timeout set in {} ms", ctx.channel(), config.getPingConnectionInterval());
                     ctx.channel().close();
                 } else {
                     sendPing(ctx);
@@ -95,7 +96,6 @@ private void sendPing(ChannelHandlerContext ctx) {
                 if (cause != null && !future.isCancelled()) {
                     log.error("Unable to send PING command over channel: " + ctx.channel(), cause);
                 }
-                log.debug("channel: {} closed due to PING response timeout set in {} ms", ctx.channel(), config.getPingConnectionInterval());
             } else {
                 sendPing(ctx);
             }