Author: remm
Date: Tue Sep 27 15:37:57 2016
New Revision: 1762505
URL: http://svn.apache.org/viewvc?rev=1762505&view=rev
Log:
Cleanup. 64*1024 ought to be enough for everybody.
Modified:
tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettingsBase.java
Modified: tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettingsBase.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettingsBase.java?rev=1762505&r1=1762504&r2=1762505&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettingsBase.java
(original)
+++ tomcat/trunk/java/org/apache/coyote/http2/ConnectionSettingsBase.java Tue
Sep 27 15:37:57 2016
@@ -35,6 +35,7 @@ public abstract class ConnectionSettings
protected static final int MIN_MAX_FRAME_SIZE = 1 << 14;
protected static final int MAX_MAX_FRAME_SIZE = (1 << 24) - 1;
protected static final long UNLIMITED = ((long)1 << 32); // Use the
maximum possible
+ protected static final int MAX_HEADER_TABLE_SIZE = 1 << 16;
// Defaults
protected static final int DEFAULT_HEADER_TABLE_SIZE = 4096;
@@ -175,8 +176,7 @@ public abstract class ConnectionSettings
private void validateHeaderTableSize(long headerTableSize) throws T {
- // Need to put a sensible limit on this. Start with 16k (default is 4k)
- if (headerTableSize > (64 * 1024)) {
+ if (headerTableSize > MAX_HEADER_TABLE_SIZE) {
String msg =
sm.getString("connectionSettings.headerTableSizeLimit",
connectionId, Long.toString(headerTableSize));
throwException(msg, Http2Error.PROTOCOL_ERROR);
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
