[AMQ-207] Timeout increment logic is incorrect

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.1
• Fix Version/s: 3.0
• Component/s: Transport

Environment

Windows

• Assignee: Unassigned
• Reporter: Neil Clayton
• Created: 05/Mar/05 15:41
• Updated: 04/Apr/05 07:00
• Resolved: 23/Mar/05 00:39

Description

The logic in CompositeTransportChannel.establishConnection appears incorrcet.

...
if (incrementTimeout && time < maximumTimeout)

{ time *= 2; time = time > maximumTimeout ? maximumTimeout : timeout; }

...

should be:

...
if (incrementTimeout && time < maximumTimeout)

{ time *= 2; time = time > maximumTimeout ? maximumTimeout : time; }

Otherwise if zeroconfig fails to get a connection, the sleep time is approx. 30s.