[asterisk-dev] [Code Review] Add reporting of call time outs to app_dial, CDR

[mjordan]

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

Review request for Asterisk Developers, otherwiseguy and may213.


Summary
-------

This patch allows app_dial to report whether or not a channel timed out using the DIALSTATUS variable.  Previously, if a channel timed out (hangup cause code AST_CAUSE_NO_USER_RESPONSE), it would be treated as a "NOANSWER".  This patch adds a new value that can be returned in the DIALSTATUS variable "TIMEDOUT", and reports the value in CDR logs.  Note that the CEL already use the actual text of the hang up cause code, so modification to them is not necessary.

Note that adding a new DIALSTATUS hangup value appeared to require some modifications to the H.323 channel drivers - hence the updates for them included in this patch.


Diffs
-----

  /trunk/CHANGES 346761 
  /trunk/addons/chan_ooh323.c 346761 
  /trunk/apps/app_dial.c 346761 
  /trunk/channels/chan_h323.c 346761 
  /trunk/channels/chan_sip.c 346761 
  /trunk/include/asterisk/cdr.h 346761 
  /trunk/main/cdr.c 346761 

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


Testing
-------

Testing was done using two Polycom SIP phones.  After registration, one of the phones would be disconnected to force a timeout when Asterisk would dial it.  When the connected phone would dial the disconnected phone, the INVITE would time out and a DIALSTATUS of TIMEDOUT was returned in the dialplan.  Testing was also done to ensure that BUSY and CONGESTION were returned in normal scenarios.

Testsuite tests will be written for this patch, both to test the DIALSTATUS variable as well as the CDR logs.


Thanks,

mjordan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20111202/1823b48c/attachment.htm>