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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
