Issue rabbitmq#42: AMQConnection missing "/" for non-default virtual hosts.

dharrigan · dharrigan · commit f3689e39bcd3 · 2015-04-22T13:49:14.000Z
If the virtual host is not the default virtual host, then there needs
to be a separator between the port and the name of the virtual host.

-=david=-
diff --git a/src/com/rabbitmq/client/impl/AMQConnection.java b/src/com/rabbitmq/client/impl/AMQConnection.java
@@ -862,7 +862,8 @@ public AMQCommand transformReply(AMQCommand command) {
     }
 
     @Override public String toString() {
-        return "amqp://" + this.username + "@" + getHostAddress() + ":" + getPort() + _virtualHost;
+        final String virtualHost = "/".equals(_virtualHost) ? _virtualHost : "/" + _virtualHost;
+        return "amqp://" + this.username + "@" + getHostAddress() + ":" + getPort() + virtualHost;
     }
 
     private String getHostAddress() {

Original file line number	Diff line number	Diff line change
`@@ -862,7 +862,8 @@ public AMQCommand transformReply(AMQCommand command) {`
`862`	`862`	`}`
`863`	`863`
`864`	`864`	`@Override public String toString() {`
`865`		`- return 5549 "amqp://" + this.username + "@" + getHostAddress() + ":" + getPort() + _virtualHost;`
	`865`	`+ final String virtualHost = "/".equals(_virtualHost) ? _virtualHost : "/" + _virtualHost;`
	`866`	`+ return "amqp://" + this.username + "@" + getHostAddress() + ":" + getPort() + virtualHost;`
`866`	`867`	`}`
`867`	`868`
`868`	`869`	`private String getHostAddress() {`