Eclipse
/
jetty.project
mirror of https://github.com/jetty/jetty.project.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixes #1878 - Handle 100 Continue response without Expect header. 

Fixed accept() to test for the response code being indeed 100.
This commit is contained in:
Simone Bordet 2017-10-09 13:38:17 +02:00
parent c69c7b29c7
commit 6fd3351272
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
jetty-client/src/main/java/org/eclipse/jetty/client/ContinueProtocolHandler.java
View File

@ -24,6 +24,7 @@ import org.eclipse.jetty.client.api.Request;
import org.eclipse.jetty.client.api.Response;
import org.eclipse.jetty.client.api.Result;
import org.eclipse.jetty.client.util.BufferingResponseListener;
import org.eclipse.jetty.http.HttpStatus;
/**
* <p>A protocol handler that handles the 100 response code.</p>
@ -49,8 +50,12 @@ public class ContinueProtocolHandler implements ProtocolHandler
@Override
public boolean accept(Request request, Response response)
{
HttpConversation conversation = ((HttpRequest)request).getConversation();
return conversation.getAttribute(ATTRIBUTE) == null;
if (response.getStatus() == HttpStatus.CONTINUE_100)
{
HttpConversation conversation = ((HttpRequest)request).getConversation();
return conversation.getAttribute(ATTRIBUTE) == null;
}
return false;
}
@Override