andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

HTTP/2 Reduce Log Level

Browse Source 
Motivation:
83c4aa6ad8 changed the log level to warn, but should have changed to debug.

Modifications:
- Change the log level to debug in Http2ConnectionHandler if the GO_AWAY fails to send. The write failure could be the result of the channel already being closed.

Result:
Fixes https://github.com/netty/netty/issues/4930.
This commit is contained in:
Scott Mitchell 2016-03-04 17:04:18 -08:00
parent c295f22390
commit 900353af52
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
codec-http2/src/main/java/io/netty/handler/codec/http2/Http2ConnectionHandler.java
View File

@ -735,8 +735,8 @@ public class Http2ConnectionHandler extends ByteToMessageDecoder implements Http
ctx.close(); ctx.close();
} }
} else { } else {
if (logger.isWarnEnabled()) { if (logger.isDebugEnabled()) {
logger.warn("{} Sending GOAWAY failed: lastStreamId '{}', errorCode '{}', " + logger.debug("{} Sending GOAWAY failed: lastStreamId '{}', errorCode '{}', " +
"debugData '{}'. Forcing shutdown of the connection.", "debugData '{}'. Forcing shutdown of the connection.",
ctx.channel(), lastStreamId, errorCode, debugData.toString(UTF_8), future.cause()); ctx.channel(), lastStreamId, errorCode, debugData.toString(UTF_8), future.cause());
} }