This is an automated email from the ASF dual-hosted git repository. remm pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit 07ba9a8ed457598a3ef4e6bdb463248524abc419 Author: remm <r...@apache.org> AuthorDate: Wed Mar 23 21:03:44 2022 +0100 Remove check disabling logging --- java/org/apache/coyote/http2/Http2Parser.java | 2 +- java/org/apache/coyote/http2/Http2UpgradeHandler.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/java/org/apache/coyote/http2/Http2Parser.java b/java/org/apache/coyote/http2/Http2Parser.java index bd91cbd..bab0ae1 100644 --- a/java/org/apache/coyote/http2/Http2Parser.java +++ b/java/org/apache/coyote/http2/Http2Parser.java @@ -310,7 +310,7 @@ class Http2Parser { int id = ByteUtil.getTwoBytes(setting, 0); long value = ByteUtil.getFourBytes(setting, 2); Setting key = Setting.valueOf(id); - if (log.isDebugEnabled() && key == Setting.UNKNOWN) { + if (key == Setting.UNKNOWN) { log.warn(sm.getString("connectionSettings.unknown", connectionId, Integer.toString(id), Long.toString(value))); } diff --git a/java/org/apache/coyote/http2/Http2UpgradeHandler.java b/java/org/apache/coyote/http2/Http2UpgradeHandler.java index a0307e4..b1119a7 100644 --- a/java/org/apache/coyote/http2/Http2UpgradeHandler.java +++ b/java/org/apache/coyote/http2/Http2UpgradeHandler.java @@ -234,7 +234,7 @@ class Http2UpgradeHandler extends AbstractStream implements InternalHttpUpgradeH int id = ByteUtil.getTwoBytes(settings, i * 6); long value = ByteUtil.getFourBytes(settings, (i * 6) + 2); Setting key = Setting.valueOf(id); - if (log.isDebugEnabled() && key == Setting.UNKNOWN) { + if (key == Setting.UNKNOWN) { log.warn(sm.getString("connectionSettings.unknown", connectionId, Integer.toString(id), Long.toString(value))); } --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org