From 8245079252865d19a929c86bfd67cbe139259f18 Mon Sep 17 00:00:00 2001 From: Johannes Dahlström Date: Mon, 16 Dec 2013 13:08:02 +0200 Subject: Decrease the websocket buffer size due to a Jetty 9.1 issue (#13087) Jetty 9.1 throws if InputBufferSize is set to greater than or equal to MaxTextMessageBufferSize. We cannot simply increase the value of the latter because Atmosphere sets the former first. Thus, its value must be set to less than 32768 (the default for MaxTextMessageBufferSize). This should not cause problems with regard to performance; the original figure of 65536 was more or less an arbitrary choice. Change-Id: If9596fc2bffdd14e8c1f31ff4b9b10d6685e01ed --- shared/src/com/vaadin/shared/communication/PushConstants.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'shared/src/com') diff --git a/shared/src/com/vaadin/shared/communication/PushConstants.java b/shared/src/com/vaadin/shared/communication/PushConstants.java index f16cbb7390..4b4f247e5f 100644 --- a/shared/src/com/vaadin/shared/communication/PushConstants.java +++ b/shared/src/com/vaadin/shared/communication/PushConstants.java @@ -27,8 +27,11 @@ public class PushConstants implements Serializable { /** * The size, in bytes, of the receiving buffer used by some servers. + *

+ * Should not be set to a value equal to or greater than 32768 due to a + * Jetty 9.1 issue (see #13087) */ - public static final int WEBSOCKET_BUFFER_SIZE = 65536; + public static final int WEBSOCKET_BUFFER_SIZE = 16384; /** * The maximum size, in characters, of a websocket message fragment. -- cgit v1.2.3