Browse Source

Drop separate user dest property for subscriptions

Before this change DefaultUserDestinationResolver provided a separate
destination prefix property for identifying "user" destinations in
subscription requests as opposed to in sent messages. Such a separate
property should not be needed.

Issue: SPR-11263
pull/432/head
Rossen Stoyanchev 12 years ago
parent
commit
abb8a93e2f
  1. 12
      spring-messaging/src/main/java/org/springframework/messaging/simp/user/DefaultUserDestinationResolver.java

12
spring-messaging/src/main/java/org/springframework/messaging/simp/user/DefaultUserDestinationResolver.java

@ -48,8 +48,6 @@ public class DefaultUserDestinationResolver implements UserDestinationResolver {
private String destinationPrefix = "/user/"; private String destinationPrefix = "/user/";
private String subscriptionDestinationPrefix = "/user/queue/";
/** /**
* Create an instance that will access user session id information through * Create an instance that will access user session id information through
@ -70,7 +68,6 @@ public class DefaultUserDestinationResolver implements UserDestinationResolver {
public void setUserDestinationPrefix(String prefix) { public void setUserDestinationPrefix(String prefix) {
Assert.hasText(prefix, "prefix must not be empty"); Assert.hasText(prefix, "prefix must not be empty");
this.destinationPrefix = prefix.endsWith("/") ? prefix : prefix + "/"; this.destinationPrefix = prefix.endsWith("/") ? prefix : prefix + "/";
this.subscriptionDestinationPrefix = this.destinationPrefix + "queue/";
} }
/** /**
@ -82,13 +79,6 @@ public class DefaultUserDestinationResolver implements UserDestinationResolver {
return this.destinationPrefix; return this.destinationPrefix;
} }
/**
* Return the prefix used to identify user destinations for (un)subscribe messages.
* <p>By default "/user/queue/".
*/
public String getSubscriptionDestinationPrefix() {
return this.subscriptionDestinationPrefix;
}
/** /**
* Return the configured {@link UserSessionRegistry}. * Return the configured {@link UserSessionRegistry}.
@ -130,7 +120,7 @@ public class DefaultUserDestinationResolver implements UserDestinationResolver {
SimpMessageType messageType = headers.getMessageType(); SimpMessageType messageType = headers.getMessageType();
if (SimpMessageType.SUBSCRIBE.equals(messageType) || SimpMessageType.UNSUBSCRIBE.equals(messageType)) { if (SimpMessageType.SUBSCRIBE.equals(messageType) || SimpMessageType.UNSUBSCRIBE.equals(messageType)) {
if (!checkDestination(destination, this.subscriptionDestinationPrefix)) { if (!checkDestination(destination, this.destinationPrefix)) {
return null; return null;
} }
if (user == null) { if (user == null) {

Loading…
Cancel
Save