Adjusted order to avoid invalid warning about TrustEverythingTrustManager

(cherry picked from commit 4398ab8)

Author: David Welsh

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

@@ -274,8 +274,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);
         if (this.amqpConnectionFactoryPostProcessor != null) {
@@ -384,6 +384,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();