]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
app_queue: Fix Queuelog EXITWITHKEY only logging two of four fields
authorMichael L. Young <elgueromexicano@gmail.com>
Sun, 6 Oct 2013 17:09:13 +0000 (17:09 +0000)
committerMichael L. Young <elgueromexicano@gmail.com>
Sun, 6 Oct 2013 17:09:13 +0000 (17:09 +0000)
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.

(closes issue ASTERISK-22197)
Reported by: Dalius M.
Tested by: Dalius M.
Patches:
    asterisk-22197-q-log-exitwithkey.diff
     uploaded by Michael L. Young (license 5026)

Review: https://reviewboard.asterisk.org/r/2901/
........

Merged revisions 400622 from http://svn.asterisk.org/svn/asterisk/branches/1.8

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@400623 65c4cc65-6c06-0410-ace0-fbb531ad65f3

apps/app_queue.c

index fef6c2053a4c19471e56af8165c05e8ad7f50b6a..a1c11a301789dfc81a6ef272ba6cc7e8a560a465 100644 (file)
@@ -7192,7 +7192,7 @@ stop:
                        }
                } else if (qe.valid_digits) {
                        ast_queue_log(args.queuename, ast_channel_uniqueid(chan), "NONE", "EXITWITHKEY",
-                               "%s|%d", qe.digits, qe.pos);
+                               "%s|%d|%d|%ld", qe.digits, qe.pos, qe.opos, (long) time(NULL) - qe.start);
                }
        }