From 61c3f455df27e7360db78517bfa5989c260d070b Mon Sep 17 00:00:00 2001 From: Artur Signell Date: Tue, 28 Apr 2015 23:50:19 +0300 Subject: [PATCH] Ensure browser does not re-use push connection after refresh (#14297, #14416) Change-Id: I05804eea5a514e84cf27214ed0dd715242cbde90 --- .../com/vaadin/server/communication/PushHandler.java | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/server/src/com/vaadin/server/communication/PushHandler.java b/server/src/com/vaadin/server/communication/PushHandler.java index ea937d279e..f14aa4826c 100644 --- a/server/src/com/vaadin/server/communication/PushHandler.java +++ b/server/src/com/vaadin/server/communication/PushHandler.java @@ -108,12 +108,18 @@ public class PushHandler extends AtmosphereResourceEventListenerAdapter { resource.getResponse().setContentType("text/plain; charset=UTF-8"); VaadinSession session = ui.getSession(); - if (resource.transport() == TRANSPORT.STREAMING) { - // Must ensure that the streaming response contains + if (resource.transport() == TRANSPORT.STREAMING + || resource.transport() == TRANSPORT.LONG_POLLING) { + // Must ensure that the streaming/long-polling response contains // "Connection: close", otherwise iOS 6 will wait for the // response to this request before sending another request to // the same server (as it will apparently try to reuse the same - // connection) + // connection). + + // Other browsers might also try to re-use the same + // connection for fetching static files after refreshing, which + // will cause a failure in loading vaadinPush.js or + // vaadinBootstrap.js resource.getResponse().addHeader("Connection", "close"); } -- 2.39.5