Merge pull request #75 from welsh/master

Adjusted order to avoid invalid warning about TrustEverythingTrustManager

src/main/java/com/rabbitmq/jms/admin/RMQConnectionFactory.java

@@ -227,8 +227,8 @@ protected Connection createConnection(String username, String password, Connecti
         this.username = username;
         this.password = password;
         com.rabbitmq.client.ConnectionFactory cf = createConnectionFactory();
-        setRabbitUri(logger, this, cf, getUri());
         maybeEnableTLS(cf);
+        setRabbitUri(logger, this, cf, getUri());
         maybeEnableHostnameVerification(cf);
         cf.setMetricsCollector(this.metricsCollector);
 
@@ -332,6 +332,7 @@ public void setUri(String uriString) throws JMSException {
         if (uriString != null && !uriString.trim().isEmpty()) {
             // Create a temp factory and set the properties by uri
             com.rabbitmq.client.ConnectionFactory factory = createConnectionFactory();
+            // Generates a TrustEverythingTrustManager warning which can be ignored as the SSLContext is not copied over
             setRabbitUri(logger, this, factory, uriString);
             // Now extract our properties from this factory, leaving the rest unchanged.
             this.host = factory.getHost();