[asterisk-dev] [Code Review] chan_sip: Manager action SIPQualifyPeer response changes

[jrose]

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

(Updated Sept. 10, 2012, 10:25 a.m.)


Review request for Asterisk Developers, Olle E Johansson and Matt Jordan.


Changes
-------

Added documentation and such.  Event is now called SIPQualifyPeerDone.  Added the triggering ActionID Header back into the event.


Summary
-------

Last week a patch was made to add some feedback for SIPQualifyPeer upon completion. Olle brought up the point that sip_poke_peer may take a considerable amount of time to resolve, so this patch adds an acknowledgement of 'will qualify' right before performing the sip_poke_peer. The response upon completion of the poke has been removed and instead an event is in place (SIPQualifyPeerComplete) to address that.

Matt mentioned in the same email thread that the event itself may not be necessary. Further discussion didn't take place, so that needs to be answered. If we decide the event is appropriate, it will need to be documented.


This addresses bug AST-969.
    https://issues.asterisk.org/jira/browse/AST-969


Diffs (updated)
-----

  /trunk/channels/chan_sip.c 372620 

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


Testing
-------

The simple stuff, ran qualify peer event and made sure the response and event went through as expected.


Thanks,

jrose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20120910/726a4a23/attachment-0001.htm>