Skip to content

Commit cd4ce87

Browse files
committed
WebSocket namespace consistently applies resolvable allowed-origins value
Issue: SPR-13760
1 parent d64ac32 commit cd4ce87

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

spring-websocket/src/main/java/org/springframework/web/socket/config/WebSocketNamespaceUtils.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,6 @@
4545
*/
4646
class WebSocketNamespaceUtils {
4747

48-
4948
public static RuntimeBeanReference registerHandshakeHandler(Element element, ParserContext context, Object source) {
5049
RuntimeBeanReference handlerRef;
5150
Element handlerElem = DomUtils.getChildElementByTagName(element, "handshake-handler");
@@ -94,7 +93,6 @@ public static RuntimeBeanReference registerSockJsService(Element element, String
9493
}
9594
else if (handshakeHandler != null) {
9695
RuntimeBeanReference handshakeHandlerRef = new RuntimeBeanReference(handshakeHandler.getAttribute("ref"));
97-
9896
RootBeanDefinition transportHandler = new RootBeanDefinition(WebSocketTransportHandler.class);
9997
transportHandler.setSource(source);
10098
transportHandler.getConstructorArgumentValues().addIndexedArgumentValue(0, handshakeHandlerRef);
@@ -106,7 +104,9 @@ else if (handshakeHandler != null) {
106104
String allowedOriginsAttribute = element.getAttribute("allowed-origins");
107105
List<String> allowedOrigins = Arrays.asList(StringUtils.tokenizeToStringArray(allowedOriginsAttribute, ","));
108106
sockJsServiceDef.getPropertyValues().add("allowedOrigins", allowedOrigins);
109-
interceptors.add(new OriginHandshakeInterceptor(allowedOrigins));
107+
RootBeanDefinition originHandshakeInterceptor = new RootBeanDefinition(OriginHandshakeInterceptor.class);
108+
originHandshakeInterceptor.getPropertyValues().add("allowedOrigins", allowedOrigins);
109+
interceptors.add(originHandshakeInterceptor);
110110
sockJsServiceDef.getPropertyValues().add("handshakeInterceptors", interceptors);
111111

112112
String attrValue = sockJsElement.getAttribute("name");
@@ -174,7 +174,7 @@ public static ManagedList<? super Object> parseBeanSubElements(Element parentEle
174174
ManagedList<? super Object> beans = new ManagedList<Object>();
175175
if (parentElement != null) {
176176
beans.setSource(context.extractSource(parentElement));
177-
for (Element beanElement : DomUtils.getChildElementsByTagName(parentElement, new String[] {"bean", "ref"})) {
177+
for (Element beanElement : DomUtils.getChildElementsByTagName(parentElement, "bean", "ref")) {
178178
beans.add(context.getDelegate().parsePropertySubElement(beanElement, null));
179179
}
180180
}

0 commit comments

Comments
 (0)