[#3945] Http2ConnectionHandler breaks channelReadComplete pipeline notification
Motivation: Http2ConnectionHandler missed to forward channelReadComplete(...) events. Modifications: Ensure we notify the next handler in the pipeline via ctx.fireChannelReadComplete(). Result: Correctly forwarding of event.
This commit is contained in:
parent
a7f83aa23e
commit
61b9da470a
@ -455,7 +455,11 @@ public class Http2ConnectionHandler extends ByteToMessageDecoder implements Http
|
|||||||
public void channelReadComplete(ChannelHandlerContext ctx) throws Exception {
|
public void channelReadComplete(ChannelHandlerContext ctx) throws Exception {
|
||||||
// Trigger flush after read on the assumption that flush is cheap if there is nothing to write and that
|
// Trigger flush after read on the assumption that flush is cheap if there is nothing to write and that
|
||||||
// for flow-control the read may release window that causes data to be written that can now be flushed.
|
// for flow-control the read may release window that causes data to be written that can now be flushed.
|
||||||
flush(ctx);
|
try {
|
||||||
|
flush(ctx);
|
||||||
|
} finally {
|
||||||
|
super.channelReadComplete(ctx);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
Loading…
Reference in New Issue
Block a user