Author: rgodfrey
Date: Tue Oct 21 09:27:15 2014
New Revision: 1633324

URL: http://svn.apache.org/r1633324
Log:
QPID-6161 : use vhost names rather than vhost node names for the name aliases

Modified:
    
qpid/trunk/qpid/java/bdbstore/systests/src/test/java/org/apache/qpid/server/store/berkeleydb/replication/GroupCreator.java
    
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostalias/VirtualHostNameAliasImpl.java

Modified: 
qpid/trunk/qpid/java/bdbstore/systests/src/test/java/org/apache/qpid/server/store/berkeleydb/replication/GroupCreator.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/bdbstore/systests/src/test/java/org/apache/qpid/server/store/berkeleydb/replication/GroupCreator.java?rev=1633324&r1=1633323&r2=1633324&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/bdbstore/systests/src/test/java/org/apache/qpid/server/store/berkeleydb/replication/GroupCreator.java
 (original)
+++ 
qpid/trunk/qpid/java/bdbstore/systests/src/test/java/org/apache/qpid/server/store/berkeleydb/replication/GroupCreator.java
 Tue Oct 21 09:27:15 2014
@@ -52,11 +52,9 @@ import org.apache.qpid.client.AMQConnect
 import org.apache.qpid.client.AMQConnectionURL;
 import org.apache.qpid.jms.ConnectionURL;
 import org.apache.qpid.server.management.plugin.HttpManagement;
-import org.apache.qpid.server.model.PatternMatchingAlias;
 import org.apache.qpid.server.model.Plugin;
 import org.apache.qpid.server.model.Port;
 import org.apache.qpid.server.model.VirtualHost;
-import org.apache.qpid.server.model.VirtualHostAlias;
 import org.apache.qpid.server.model.VirtualHostNode;
 import org.apache.qpid.server.virtualhost.berkeleydb.BDBHAVirtualHostImpl;
 import org.apache.qpid.server.virtualhostnode.AbstractVirtualHostNode;
@@ -160,17 +158,6 @@ public class GroupCreator
             brokerConfiguration.setObjectAttribute(Plugin.class, 
TestBrokerConfiguration.ENTRY_NAME_HTTP_MANAGEMENT, 
HttpManagement.HTTP_BASIC_AUTHENTICATION_ENABLED, true);
             brokerConfiguration.setObjectAttribute(Port.class, 
TestBrokerConfiguration.ENTRY_NAME_HTTP_PORT, Port.PORT, 
_testcase.getHttpManagementPort(brokerPort));
 
-            final Map<String, Object> aliasAttributes  = new HashMap<>();
-            aliasAttributes.put(VirtualHostAlias.NAME, "testAlias");
-            aliasAttributes.put(VirtualHostAlias.TYPE, 
PatternMatchingAlias.TYPE_NAME);
-            aliasAttributes.put(PatternMatchingAlias.PATTERN, "test");
-            aliasAttributes.put(PatternMatchingAlias.VIRTUAL_HOST_NODE, 
nodeName);
-            brokerConfiguration.addObjectConfiguration(Port.class,
-                                                       
TestBrokerConfiguration.ENTRY_NAME_AMQP_PORT,
-                                                       VirtualHostAlias.class,
-                                                       aliasAttributes);
-
-
             brokerConfiguration.setObjectAttributes(VirtualHostNode.class, 
_virtualHostName, virtualHostNodeAttributes);
 
         }

Modified: 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostalias/VirtualHostNameAliasImpl.java
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostalias/VirtualHostNameAliasImpl.java?rev=1633324&r1=1633323&r2=1633324&view=diff
==============================================================================
--- 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostalias/VirtualHostNameAliasImpl.java
 (original)
+++ 
qpid/trunk/qpid/java/broker-core/src/main/java/org/apache/qpid/server/virtualhostalias/VirtualHostNameAliasImpl.java
 Tue Oct 21 09:27:15 2014
@@ -25,6 +25,7 @@ import java.util.Map;
 import org.apache.qpid.server.model.Broker;
 import org.apache.qpid.server.model.ManagedObjectFactoryConstructor;
 import org.apache.qpid.server.model.Port;
+import org.apache.qpid.server.model.VirtualHost;
 import org.apache.qpid.server.model.VirtualHostNameAlias;
 import org.apache.qpid.server.model.VirtualHostNode;
 
@@ -44,7 +45,8 @@ public final class VirtualHostNameAliasI
         Broker<?> broker = getPort().getParent(Broker.class);
         for(VirtualHostNode<?> vhn : broker.getVirtualHostNodes())
         {
-            if(vhn.getName().equals(name))
+            VirtualHost<?, ?, ?> vh = vhn.getVirtualHost();
+            if(vh != null && vh.getName().equals(name))
             {
                 return vhn;
             }



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to