[asterisk-bugs] [Asterisk 0013239]: [patch] Queue timeouts not always working correctly.

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13239 
====================================================================== 
Reported By:                atis
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13239
Category:                   Applications/app_queue
Reproducibility:            sometimes
Severity:                   minor
Priority:                   normal
Status:                     feedback
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 135597 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-08-05 09:41 CDT
Last Modified:              2008-08-14 11:20 CDT
====================================================================== 
Summary:                    [patch] Queue timeouts not always working correctly.
Description: 
This is follow-up of issue http://bugs.digium.com/view.php?id=13186 which
inspired my search greatly. 

I observed that queue timeout doesn't always goes out after time specified
in Queue() args. 

Sample case - when Queue(x,,,,10) is called, and retry is set to 5, and
agent ring time is 10 seconds, queue will ring agent for 10 seconds, and
sometimes wait 5 more seconds. This is because timeout checks for
greater-than, but not for equal value. 

Also there is additional check for expiry within ring_one, as for very
specific configuration with custom strategy it doesn't goes out from there.
I'm not sure if this is affecting any other strategy, but perhaps it could
be repeated on some ringall configuration.


====================================================================== 

---------------------------------------------------------------------- 
 (0091408) atis (reporter) - 2008-08-14 11:20
 http://bugs.digium.com/view.php?id=13239#c91408 
---------------------------------------------------------------------- 
Ok, i found that this was actually necessary for timeoutpriority=conf
within issue 12690

Attached clean patch that fixes only this issue. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-08-14 11:20 atis           Note Added: 0091408                          
======================================================================