From 2988fb8eeb7cc136b6b7660bcc5774cccc313744 Mon Sep 17 00:00:00 2001 From: Norman Maurer Date: Sun, 30 Jul 2017 07:33:17 +0200 Subject: [PATCH] Ensure Http2FrameCodec uses Http2Settings.defaultSettings() Motivation: Http2FrameCodec should use Http2Settings.defaultSettings() when no Http2Settings were specified by the user. Modifications: Replace new Http2Settings() with Http2Settings.defaultSettings() Result: Use correct Http2Settings by default when using Http2FrameCodec in all cases. --- .../main/java/io/netty/handler/codec/http2/Http2FrameCodec.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2FrameCodec.java b/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2FrameCodec.java index d49fd92198..8525388897 100644 --- a/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2FrameCodec.java +++ b/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2FrameCodec.java @@ -127,7 +127,7 @@ public class Http2FrameCodec extends ChannelDuplexHandler { * @param server {@code true} this is a server */ public Http2FrameCodec(boolean server, Http2FrameLogger frameLogger) { - this(server, new DefaultHttp2FrameWriter(), frameLogger, new Http2Settings()); + this(server, new DefaultHttp2FrameWriter(), frameLogger, Http2Settings.defaultSettings()); } // Visible for testing