[QPIDJMS-567] The FailoverProvider does not report the root cause of a connection attempt failure

• Type: Wish
• Status: Closed
• Priority: Major
• Resolution: Information Provided
• Component/s: qpid-jms-client
• Assignee: Unassigned
• Reporter: Philippe Haution
• Created: 22/Mar/22 18:11
• Updated: 07/Jul/22 18:57
• Resolved: 07/Jul/22 18:57

Description

In the FailoverProvider.triggerReconnectionAttempt() method, in case the connection attempt fails, we end up in a catch block where the Throwable that explains the problem is not logged to the user.

Instead of :

catch (Throwable e) {
  LOG.info("Connection attempt:[{}] to: {} failed", reconnectAttempts,
           target.getScheme() + "://" + target.getHost() + ":" + target.getPort());

We should have:

catch (Throwable e) {
  LOG.info("Connection attempt:[{}] to: {} failed", reconnectAttempts,
           target.getScheme() + "://" + target.getHost() + ":" + target.getPort(), e);