Apache/activemq
1
0
Fork 0
mirror of https://github.com/apache/activemq.git synced 2025-02-16 23:16:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

https://issues.apache.org/jira/browse/AMQ-6304

Browse Source 
Ensure that the broker send a container ID using its assigned Broker
name value.
(cherry picked from commit 76b70545f248e6970245719f2f584298b8d493fd)
This commit is contained in:
Timothy Bish 2016-06-01 18:38:41 -04:00
parent 6d539b4f01
commit ceeb1f68ac
2 changed files with 30 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
activemq-amqp/src/main/java/org/apache/activemq/transport/amqp/protocol/AmqpConnection.java
View File

@ -492,6 +492,7 @@ public class AmqpConnection implements AmqpProtocolConverter {
protonConnection.setOfferedCapabilities(getConnectionCapabilitiesOffered());
protonConnection.setProperties(getConnetionProperties());
protonConnection.setContainer(brokerService.getBrokerName());
protonConnection.open();
configureInactivityMonitor();

29
activemq-amqp/src/test/java/org/apache/activemq/transport/amqp/interop/AmqpConnectionsTest.java
View File

@ -113,6 +113,35 @@ public class AmqpConnectionsTest extends AmqpClientTestSupport {
assertEquals(0, getProxyToBroker().getCurrentConnectionsCount());
}
@Test(timeout = 60000)
public void testConnectionCarriesContainerId() throws Exception {
AmqpClient client = createAmqpClient();
assertNotNull(client);
client.setValidator(new AmqpValidator() {
@Override
public void inspectOpenedResource(Connection connection) {
String remoteContainer = connection.getRemoteContainer();
if (remoteContainer == null || !remoteContainer.equals(brokerService.getBrokerName())) {
markAsInvalid("Broker did not send a valid container ID");
} else {
LOG.info("Broker container ID = {}", remoteContainer);
}
}
});
AmqpConnection connection = client.connect();
assertNotNull(connection);
assertEquals(1, getProxyToBroker().getCurrentConnectionsCount());
connection.getStateInspector().assertValid();
connection.close();
assertEquals(0, getProxyToBroker().getCurrentConnectionsCount());
}
@Test(timeout = 60000)
public void testCanConnectWithDifferentContainerIds() throws Exception {
AmqpClient client = createAmqpClient();