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

HeadContext is inbound and outbound (#8592)

Browse Source 
Motivation:

Our HeadContext in DefaultChannelPipeline does handle inbound and outbound but we only marked it as outbound. While this does not have any effect in the current code-base it can lead to problems when we change our internals (this is also how I found the bug).

Modifications:

Construct HeadContext so it is also marked as handling inbound.

Result:

More correct code.
This commit is contained in:
Norman Maurer 2018-11-24 10:47:56 +01:00 committed by GitHub
parent 2a2bc21067
commit af34287fd1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
transport/src/main/java/io/netty/channel/DefaultChannelPipeline.java
View File

@ -1331,7 +1331,7 @@ public class DefaultChannelPipeline implements ChannelPipeline {
private final Unsafe unsafe; private final Unsafe unsafe;
HeadContext(DefaultChannelPipeline pipeline) { HeadContext(DefaultChannelPipeline pipeline) {
super(pipeline, null, HEAD_NAME, false, true); super(pipeline, null, HEAD_NAME, true, true);
unsafe = pipeline.channel().unsafe(); unsafe = pipeline.channel().unsafe();
setAddComplete(); setAddComplete();
} }