Skip to content

Commit 1ad0edc

Browse files
committed
Expect:100-continue fixes for Netty
Signed-off-by: Maxim Nesen <maxim.nesen@oracle.com>
1 parent 97a2ec2 commit 1ad0edc

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

connectors/netty-connector/src/main/java/org/glassfish/jersey/netty/connector/JerseyExpectContinueHandler.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,8 @@ public void channelRead(ChannelHandlerContext ctx, Object msg) throws Exception
5454
}
5555
if (!HttpResponseStatus.CONTINUE.equals(response.status())) {
5656
ctx.fireChannelRead(msg); //bypass the message to the next handler in line
57+
} else {
58+
ctx.pipeline().remove(this);
5759
}
5860
} else {
5961
if (!isExpected) {

docs/src/main/docbook/appendix-properties.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2041,9 +2041,9 @@
20412041
<entry><literal>jersey.config.client.request.expect.100.continue.timeout</literal></entry>
20422042
<entry>
20432043
<para>
2044-
Property for expect:100-continue timeout value which shall be waited until the full
2045-
request is sent ignoring Expect:100-continue feature.
2046-
Default timeout value is (500 ms) after which Expect:100-Continue feature is ignored.
2044+
This timeout is used for waiting for 100-Continue response when 100-Continue
2045+
is sent by the client.
2046+
Default timeout value is 500 ms after which Expect:100-Continue feature is ignored.
20472047
<literal>Since 2.41</literal>
20482048
</para>
20492049
</entry>

0 commit comments

Comments
 (0)