Issue #1676 remove deprecated HttpParser STRICT system property
This commit is contained in:
parent
a53222cc72
commit
d14c53ed30
|
@ -25,6 +25,7 @@ import java.util.concurrent.ConcurrentHashMap;
|
|||
|
||||
import org.eclipse.jetty.fcgi.FCGI;
|
||||
import org.eclipse.jetty.http.BadMessageException;
|
||||
import org.eclipse.jetty.http.HttpCompliance;
|
||||
import org.eclipse.jetty.http.HttpField;
|
||||
import org.eclipse.jetty.http.HttpFields;
|
||||
import org.eclipse.jetty.http.HttpHeader;
|
||||
|
@ -329,7 +330,7 @@ public class ResponseContentParser extends StreamContentParser
|
|||
{
|
||||
private FCGIHttpParser(ResponseHandler handler)
|
||||
{
|
||||
super(handler, 65 * 1024, true);
|
||||
super(handler, 65 * 1024, HttpCompliance.LEGACY);
|
||||
reset();
|
||||
}
|
||||
|
||||
|
|
|
@ -81,7 +81,7 @@ import static org.eclipse.jetty.http.HttpTokens.TAB;
|
|||
* <dl>
|
||||
* <dt>RFC7230</dt><dd>(default) Compliance with RFC7230</dd>
|
||||
* <dt>RFC2616</dt><dd>Wrapped headers and HTTP/0.9 supported</dd>
|
||||
* <dt>LEGACY</dt><dd>(aka STRICT) Adherence to Servlet Specification requirement for
|
||||
* <dt>LEGACY</dt><dd>Adherence to Servlet Specification requirement for
|
||||
* exact case of header names, bypassing the header caches, which are case insensitive,
|
||||
* otherwise equivalent to RFC2616</dd>
|
||||
* </dl>
|
||||
|
@ -90,8 +90,6 @@ import static org.eclipse.jetty.http.HttpTokens.TAB;
|
|||
public class HttpParser
|
||||
{
|
||||
public static final Logger LOG = Log.getLogger(HttpParser.class);
|
||||
@Deprecated
|
||||
public final static String __STRICT="org.eclipse.jetty.http.HttpParser.STRICT";
|
||||
public final static int INITIAL_URI_LENGTH=256;
|
||||
|
||||
/**
|
||||
|
@ -236,8 +234,7 @@ public class HttpParser
|
|||
|
||||
private static HttpCompliance compliance()
|
||||
{
|
||||
Boolean strict = Boolean.getBoolean(__STRICT);
|
||||
return strict?HttpCompliance.LEGACY:HttpCompliance.RFC7230;
|
||||
return HttpCompliance.RFC7230;
|
||||
}
|
||||
|
||||
/* ------------------------------------------------------------------------------- */
|
||||
|
@ -264,20 +261,6 @@ public class HttpParser
|
|||
this(handler,maxHeaderBytes,compliance());
|
||||
}
|
||||
|
||||
/* ------------------------------------------------------------------------------- */
|
||||
@Deprecated
|
||||
public HttpParser(RequestHandler handler,int maxHeaderBytes,boolean strict)
|
||||
{
|
||||
this(handler,maxHeaderBytes,strict?HttpCompliance.LEGACY:compliance());
|
||||
}
|
||||
|
||||
/* ------------------------------------------------------------------------------- */
|
||||
@Deprecated
|
||||
public HttpParser(ResponseHandler handler,int maxHeaderBytes,boolean strict)
|
||||
{
|
||||
this(handler,maxHeaderBytes,strict?HttpCompliance.LEGACY:compliance());
|
||||
}
|
||||
|
||||
/* ------------------------------------------------------------------------------- */
|
||||
public HttpParser(RequestHandler handler,HttpCompliance compliance)
|
||||
{
|
||||
|
|
Loading…
Reference in New Issue