[asterisk-dev] [Code Review] 2901: Fix Regression With Queuelog EXITWITHKEY Only Logging Two Out Of Four Fields

svnbot reviewboard at asterisk.org
Sun Oct 6 12:07:33 CDT 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2901/
-----------------------------------------------------------

(Updated Oct. 6, 2013, 12:07 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 400622


Bugs: ASTERISK-22197
    https://issues.asterisk.org/jira/browse/ASTERISK-22197


Repository: Asterisk


Description
-------

Commit r62462 added two extra fields for logging "the original position the caller entered the queue at, and the amount of time the caller was waiting in the queue."  But when r75969 was merged from 1.4 into trunk (r75977), these two fields disappeared. Those two extra fields were not logged in 1.4 and when the patch was merged, those fields went away.

Therefore, this is a regression and was caught by the reporter because he was reading the awesome "Asterisk: The Definitive Guide" book.


Diffs
-----

  /branches/1.8/apps/app_queue.c 400452 

Diff: https://reviewboard.asterisk.org/r/2901/diff/


Testing
-------

Reporter tested the patch and confirmed that it restored the logging of these two fields.


Thanks,

Michael Young

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20131006/4adf3316/attachment.html>


More information about the asterisk-dev mailing list