Fix potential invalid JSON output in MetadataWriter (#14591)

Change-Id: I90ec428ae11727b9762d53ebb8d94f8c850c425b

diff --git a/server/src/com/vaadin/server/communication/MetadataWriter.java b/server/src/com/vaadin/server/communication/MetadataWriter.java
index 17c893fe4076c0bc777284b4d61f9d45637f499f..9c46e646926bcd6af675aeac51d4f66af8cb2a37 100644 (file)
--- a/server/src/com/vaadin/server/communication/MetadataWriter.java
+++ b/server/src/com/vaadin/server/communication/MetadataWriter.java
@@ -70,6 +70,7 @@ public class MetadataWriter implements Serializable {
             if (metaOpen) {
                 writer.write(", ");
             }
+            metaOpen = true;
             writer.write("\"async\":true");
         }
 

diff --git a/server/tests/src/com/vaadin/server/communication/MetadataWriterTest.java b/server/tests/src/com/vaadin/server/communication/MetadataWriterTest.java
index dee37ddc7f615704aadf42f107591f3fed3c5b10..afae14007e18951b28e7383f1c27579001f8578c 100644 (file)
--- a/server/tests/src/com/vaadin/server/communication/MetadataWriterTest.java
+++ b/server/tests/src/com/vaadin/server/communication/MetadataWriterTest.java
@@ -18,6 +18,7 @@ package com.vaadin.server.communication;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
+import java.io.IOException;
 import java.io.StringWriter;
 
 import org.junit.Assert;
@@ -92,4 +93,17 @@ public class MetadataWriterTest {
                 "{\"timedRedirect\":{\"interval\":15,\"url\":\"\"}}", writer
                         .getBuffer().toString());
     }
+
+    @Test
+    public void writeAsyncWithSystemMessages() throws IOException {
+        WrappedSession wrappedSession = mock(WrappedSession.class);
+        when(session.getSession()).thenReturn(wrappedSession);
+
+        disableSessionExpirationMessages(messages);
+
+        new MetadataWriter().write(ui, writer, false, true, messages);
+        Assert.assertEquals(
+                "{\"async\":true,\"timedRedirect\":{\"interval\":15,\"url\":\"\"}}",
+                writer.getBuffer().toString());
+    }
 }