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

Revert "Issue #346 HttpParser RFC2616 Compliance mode" 

This reverts commit 58e5a54ab6.
RFC7430 allows empty header values
This commit is contained in:
Greg Wilkins 2016-03-05 11:03:32 +01:00
parent 58e5a54ab6
commit d1fc5eb67d
4 changed files with 10 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
jetty-http/src/main/java/org/eclipse/jetty/http/HttpParser.java
View File

@ -33,7 +33,6 @@ import org.eclipse.jetty.util.TypeUtil;
import org.eclipse.jetty.util.Utf8StringBuilder;
import org.eclipse.jetty.util.log.Log;
import org.eclipse.jetty.util.log.Logger;
import org.omg.CosNaming._NamingContextExtStub;
import static org.eclipse.jetty.http.HttpTokens.CARRIAGE_RETURN;
import static org.eclipse.jetty.http.HttpTokens.LINE_FEED;
@ -1175,12 +1174,8 @@ public class HttpParser
{
_value=null;
_string.setLength(0);
_valueString=null;
_valueString=_compliance.ordinal()<=HttpCompliance.RFC2616.ordinal()?"":null;
_length=-1;
if (_compliance.ordinal()<=HttpCompliance.RFC2616.ordinal())
_valueString="";
else
_headerString=null;
setState(State.HEADER);
break;

10
jetty-http/src/test/java/org/eclipse/jetty/http/HttpParserTest.java
View File

@ -388,9 +388,13 @@ public class HttpParserTest
assertEquals("HTTP/1.0", _versionOrReason);
assertEquals("Host", _hdr[0]);
assertEquals("localhost", _val[0]);
assertEquals("Connection", _hdr[1]);
assertEquals("close", _val[1]);
assertEquals(1, _headers);
assertEquals("Name0", _hdr[1]);
assertEquals(null, _val[1]);
assertEquals("Name1", _hdr[2]);
assertEquals(null, _val[2]);
assertEquals("Connection", _hdr[3]);
assertEquals("close", _val[3]);
assertEquals(3, _headers);
}
@Test

2
jetty-http2/http2-client/src/test/java/org/eclipse/jetty/http2/client/PrefaceTest.java
View File

@ -236,7 +236,7 @@ public class PrefaceTest extends AbstractTest
"Host: localhost\r\n" +
"Connection: Upgrade, HTTP2-Settings\r\n" +
"Upgrade: h2c\r\n" +
"HTTP2-Settings: ==\r\n" +
"HTTP2-Settings: \r\n" +
"\r\n";
ByteBuffer upgradeBuffer = ByteBuffer.wrap(upgradeRequest.getBytes(StandardCharsets.ISO_8859_1));
socket.write(upgradeBuffer);

2
jetty-http2/http2-server/src/test/java/org/eclipse/jetty/http2/server/HTTP2CServerTest.java
View File

@ -123,7 +123,7 @@ public class HTTP2CServerTest extends AbstractServerTest
"Host: localhost\r\n" +
"Connection: something, else, upgrade, HTTP2-Settings\r\n" +
"Upgrade: h2c\r\n" +
"HTTP2-Settings: ==\r\n" +
"HTTP2-Settings: \r\n" +
"\r\n").getBytes(StandardCharsets.ISO_8859_1));
output.flush();