Use correct header for version in CONNECTED frame

wilkinsona · rstoyanchev · commit 364bc357095e · 2013-08-29T09:58:41.000-04:00
Previously, the negotiated version was included in the CONNECTED frame
using the accept-version header. This is incorrect. The version
header should be used.
diff --git a/spring-messaging/src/main/java/org/springframework/messaging/simp/stomp/StompProtocolHandler.java b/spring-messaging/src/main/java/org/springframework/messaging/simp/stomp/StompProtocolHandler.java
@@ -185,10 +185,10 @@ protected void handleConnect(WebSocketSession session, Message<?> message) throw
 
 		Set<String> acceptVersions = connectHeaders.getAcceptVersion();
 		if (acceptVersions.contains("1.2")) {
-			connectedHeaders.setAcceptVersion("1.2");
+			connectedHeaders.setVersion("1.2");
 		}
 		else if (acceptVersions.contains("1.1")) {
-			connectedHeaders.setAcceptVersion("1.1");
+			connectedHeaders.setVersion("1.1");
 		}
 		else if (acceptVersions.isEmpty()) {
 			// 1.0

Original file line number	Diff line number	Diff line change
`@@ -185,10 +185,10 @@ protected void handleConnect(WebSocketSession session, Message<?> message) throw`
`185`	`185`
`186`	`186`	`Set<String> acceptVersions = connectHeaders.getAcceptVersion();`
`187`	`187`	`if (acceptVersions.contains("1.2")) {`
`188`		`- connectedHeaders.setAcceptVersion("1.2");`
	`188`	`+ connectedHeaders.setVersion("1.2");`
`189`	`189`	`}`
`190`	`190`	`else if (acceptVersions.contains("1.1")) {`
`191`		`- connectedHeaders.setAcceptVersion("1.1");`
	`191`	`+ connectedHeaders.setVersion("1.1");`
`192`	`192`	`}`
`193`	`193`	`else if (acceptVersions.isEmpty()) {`
`194`	`194`	`// 1.0`