[AMQ-2511] Regression: Inactivity monitor does not time out stale connections.

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.3.0
• Fix Version/s: 5.3.2, 5.4.0
• Component/s: Broker
• Regression: Regression
• Assignee: Hiram R. Chirino
• Reporter: Hiram R. Chirino
• Created: 25/Nov/09 17:22
• Updated: 14/May/10 08:41
• Resolved: 14/May/10 08:41

Description

The fix @ AMQ-2088 cause a the Inactivity Monitor to regress.

Now when a socket is waiting to read the next message, or is in the middle of receiving the next message it sets a flag that disable the inactivity monitor from timing out the connection. The problem is that if the connection gets hung, then it won't time out since the flag is set.

I think a better approach would be for the wireformat to expose a counter which is incremented as data is read.

That way between inactivity check intervals the inactivity monitor can check to see if that value has changed since it's last check.

Issue Links

• is broken by: AMQ-2088 Change the InactivityMonitor to clear its flag as soon as a few bytes are received on a connection rather than only after an entire message has been assembled.