Remove debug log to avoid potential String decode issue

2011aad · Jan 12, 2021 · e62d7bb · e62d7bb
1 parent 3e63ea3
commit e62d7bb
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/main/java/ru/yandex/clickhouse/jdbcbridge/JdbcBridgeVerticle.java b/src/main/java/ru/yandex/clickhouse/jdbcbridge/JdbcBridgeVerticle.java
@@ -286,9 +286,10 @@ private void responseHandlers(RoutingContext ctx) {
             log.debug("[{}] Parameters:\n{}", path, req.params());
         }
 
-        if (log.isTraceEnabled()) {
-            log.trace("[{}] Body:\n{}", path, ctx.getBodyAsString());
-        }
+        // bad assumption here as it may lead to UTF8 decode issue like #83
+        // if (log.isTraceEnabled()) {
+        // log.trace("[{}] Body:\n{}", path, ctx.getBodyAsString());
+        // }
 
         HttpServerResponse resp = ctx.response();