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

Review request for Asterisk Developers.


Repository: Asterisk


Description
-------

This corrects two issues with the extra field information in Asterisk 12+ in 
channel event logs.

It is possible to inject custom values into the dialstatus provided by 
ast_channel_dial_type() Stasis messages that fall outside the enumeration 
allowed for the DIALSTATUS channel variable. CEL now filters for the allowed 
values and ignores other values.

The "hangupsource" extra field key is always blank if the far end channel is a 
chan_pjsip channel. This is because the hangupsource is never set for the pjsip 
channel driver. This change sets the hangupsource whenever a hangup is queued 
for chan_pjsip channels.


Diffs
-----

  branches/12/tests/test_cel.c 417545 
  branches/12/res/res_pjsip_session.c 417545 
  branches/12/main/cel.c 417545 
  branches/12/channels/chan_pjsip.c 417545 

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


Testing
-------

Tested both portions of the patch manually and the dial status portion using 
the included unit test.


Thanks,

opticron

-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Reply via email to