diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/support/AbstractSockJsService.java b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/support/AbstractSockJsService.java index 196330b558..0f30ca9868 100644 --- a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/support/AbstractSockJsService.java +++ b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/support/AbstractSockJsService.java @@ -614,22 +614,23 @@ public abstract class AbstractSockJsService implements SockJsService, CorsConfig private class IframeHandler implements SockJsRequestHandler { private static final String IFRAME_CONTENT = - "\n" + - "\n" + - "\n" + - " \n" + - " \n" + - " \n" + - " \n" + - "\n" + - "\n" + - "

Don't panic!

\n" + - "

This is a SockJS hidden iframe. It's used for cross domain magic.

\n" + - "\n" + - ""; + """ + + + + + + + + + +

Don't panic!

+

This is a SockJS hidden iframe. It's used for cross domain magic.

+ + """; @Override public void handle(ServerHttpRequest request, ServerHttpResponse response) throws IOException { diff --git a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/transport/handler/HtmlFileTransportHandler.java b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/transport/handler/HtmlFileTransportHandler.java index 13b09e2cb1..cc72de2276 100644 --- a/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/transport/handler/HtmlFileTransportHandler.java +++ b/spring-websocket/src/main/java/org/springframework/web/socket/sockjs/transport/handler/HtmlFileTransportHandler.java @@ -60,18 +60,19 @@ public class HtmlFileTransportHandler extends AbstractHttpSendingTransportHandle static { StringBuilder sb = new StringBuilder( - "\n" + - "\n" + - " \n" + - " \n" + - "

Don't panic!

\n" + - " " + """ + + + + +

Don't panic!

+ """ ); while (sb.length() < MINIMUM_PARTIAL_HTML_CONTENT_LENGTH) { diff --git a/spring-websocket/src/test/java/org/springframework/web/socket/messaging/StompSubProtocolHandlerTests.java b/spring-websocket/src/test/java/org/springframework/web/socket/messaging/StompSubProtocolHandlerTests.java index 6744cbc3d8..a9932bfb99 100644 --- a/spring-websocket/src/test/java/org/springframework/web/socket/messaging/StompSubProtocolHandlerTests.java +++ b/spring-websocket/src/test/java/org/springframework/web/socket/messaging/StompSubProtocolHandlerTests.java @@ -109,7 +109,11 @@ public class StompSubProtocolHandlerTests { assertThat(this.session.getSentMessages().size()).isEqualTo(1); WebSocketMessage textMessage = this.session.getSentMessages().get(0); - assertThat(textMessage.getPayload()).isEqualTo(("CONNECTED\n" + "user-name:joe\n" + "\n" + "\u0000")); + assertThat(textMessage.getPayload()).isEqualTo((""" + CONNECTED + user-name:joe + + \u0000""")); } @Test @@ -123,7 +127,11 @@ public class StompSubProtocolHandlerTests { assertThat(this.session.getSentMessages().size()).isEqualTo(1); WebSocketMessage textMessage = this.session.getSentMessages().get(0); - assertThat(textMessage.getPayload()).isEqualTo(("CONNECTED\n" + "user-name:joe\n" + "\n" + "\u0000")); + assertThat(textMessage.getPayload()).isEqualTo((""" + CONNECTED + user-name:joe + + \u0000""")); } @Test @@ -142,8 +150,13 @@ public class StompSubProtocolHandlerTests { assertThat(this.session.getSentMessages().size()).isEqualTo(1); TextMessage actual = (TextMessage) this.session.getSentMessages().get(0); - assertThat(actual.getPayload()).isEqualTo(("CONNECTED\n" + "version:1.2\n" + "heart-beat:15000,15000\n" + - "user-name:joe\n" + "\n" + "\u0000")); + assertThat(actual.getPayload()).isEqualTo((""" + CONNECTED + version:1.2 + heart-beat:15000,15000 + user-name:joe + + \u0000""")); } @Test @@ -161,8 +174,13 @@ public class StompSubProtocolHandlerTests { assertThat(this.session.getSentMessages().size()).isEqualTo(1); TextMessage actual = (TextMessage) this.session.getSentMessages().get(0); - assertThat(actual.getPayload()).isEqualTo(("CONNECTED\n" + "version:1.0\n" + "heart-beat:0,0\n" + - "user-name:joe\n" + "\n" + "\u0000")); + assertThat(actual.getPayload()).isEqualTo((""" + CONNECTED + version:1.0 + heart-beat:0,0 + user-name:joe + + \u0000""")); } @Test @@ -178,8 +196,12 @@ public class StompSubProtocolHandlerTests { assertThat(this.session.getSentMessages().size()).isEqualTo(1); TextMessage actual = (TextMessage) this.session.getSentMessages().get(0); - assertThat(actual.getPayload()).isEqualTo(("ERROR\n" + "message:Session closed.\n" + "content-length:0\n" + - "\n\u0000")); + assertThat(actual.getPayload()).isEqualTo((""" + ERROR + message:Session closed. + content-length:0 + + \u0000""")); } @Test @@ -196,7 +218,11 @@ public class StompSubProtocolHandlerTests { assertThat(this.session.getSentMessages().size()).isEqualTo(1); TextMessage actual = (TextMessage) this.session.getSentMessages().get(0); - assertThat(actual.getPayload()).isEqualTo(("RECEIPT\n" + "receipt-id:message-123\n" + "\n\u0000")); + assertThat(actual.getPayload()).isEqualTo((""" + RECEIPT + receipt-id:message-123 + + \u0000""")); } @Test @@ -391,7 +417,11 @@ public class StompSubProtocolHandlerTests { assertThat(this.session.getSentMessages()).hasSize(1); WebSocketMessage textMessage = this.session.getSentMessages().get(0); assertThat(textMessage.getPayload()) - .isEqualTo("CONNECTED\n" + "user-name:__pete__@gmail.com\n" + "\n" + "\u0000"); + .isEqualTo(""" + CONNECTED + user-name:__pete__@gmail.com + + \u0000"""); } @Test @@ -470,7 +500,11 @@ public class StompSubProtocolHandlerTests { assertThat(this.session.getSentMessages().size()).isEqualTo(1); textMessage = (TextMessage) this.session.getSentMessages().get(0); - assertThat(textMessage.getPayload()).isEqualTo(("CONNECTED\n" + "user-name:joe\n" + "\n" + "\u0000")); + assertThat(textMessage.getPayload()).isEqualTo((""" + CONNECTED + user-name:joe + + \u0000""")); this.protocolHandler.afterSessionEnded(this.session, CloseStatus.BAD_DATA, this.channel); diff --git a/spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransportTests.java b/spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransportTests.java index ea13095842..ed6e51b95b 100644 --- a/spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransportTests.java +++ b/spring-websocket/src/test/java/org/springframework/web/socket/sockjs/client/RestTemplateXhrTransportTests.java @@ -75,7 +75,10 @@ public class RestTemplateXhrTransportTests { @Test public void connectReceiveAndClose() throws Exception { - String body = "o\n" + "a[\"foo\"]\n" + "c[3000,\"Go away!\"]"; + String body = """ + o + a["foo"] + c[3000,"Go away!"]"""; ClientHttpResponse response = response(HttpStatus.OK, body); connect(response); @@ -157,7 +160,11 @@ public class RestTemplateXhrTransportTests { @Test public void responseClosedAfterDisconnected() throws Exception { - String body = "o\n" + "c[3000,\"Go away!\"]\n" + "a[\"foo\"]\n"; + String body = """ + o + c[3000,"Go away!"] + a["foo"] + """; ClientHttpResponse response = response(HttpStatus.OK, body); connect(response);