[GitHub] [activemq-artemis] gemmellr commented on a change in pull request #3720: NO-JIRA Fix broker connection AMQP SSL example

[GitBox <gi...@apache.org>]

gemmellr commented on a change in pull request #3720:
URL: https://github.com/apache/activemq-artemis/pull/3720#discussion_r701214740



##########
File path: examples/features/broker-connection/amqp-sending-overssl/src/main/java/org/apache/activemq/artemis/jms/example/BrokerConnectionSenderSSL.java
##########
@@ -58,7 +58,7 @@ public static void main(final String[] args) throws Exception {
       // Step 2. create a connection on server1, and receive a few messages.
       //          the sender on the broker conneciton will take care of the transfer.
       Connection connectionOnServer1 = null;
-      ConnectionFactory connectionFactoryServer1 = new JmsConnectionFactory("amqps://localhost:5771?transport.trustStoreLocation=target/server1/etc/activemq.example.truststore&transport.trustStorePassword=activemqexample");
+      ConnectionFactory connectionFactoryServer1 = new JmsConnectionFactory("amqps://localhost:5771?transport.trustStoreLocation=target/server0/etc/server-ca-truststore.jks&transport.trustStorePassword=securepass");

Review comment:
       Why change it to using the 'server0' ca trust store file when it is connecting to server1 ? (I expect they are copies of the same file currently, but regardless...if they both exist it should use the 'matching' one)




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org