[asterisk-dev] [Code Review] 2494: Add multi-channel blob message type; refactor ChanSpyStart/ChanSpyStop events to use it

Matt Jordan reviewboard at asterisk.org
Mon May 6 17:48:56 CDT 2013


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

(Updated May 6, 2013, 10:48 p.m.)


Review request for Asterisk Developers.


Changes
-------

Addressed David's findings.


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


Repository: Asterisk


Description
-------

Multi-channel blob messages were added awhile back for Dial; however, the multi-channel blob is used by Dial not as a message type but rather as a payload. This patch gives multi-channel blobs their own generic message type for the random kinds of things we're going to need to use it for (ChanSpy being the first candidate).

I've left Dial alone even though its payload now has its own generic message type. Dial messages have a significant number of producers and consumers, in a variety of modules. (app_dial, app_queue, app_followme, pbx, cdr, manager_channels, etc. - the list will grow) It feels important enough that having its own message type is useful and avoids the extra step of parsing out the JSON 'type' field.


Diffs (updated)
-----

  /trunk/apps/app_chanspy.c 387741 
  /trunk/include/asterisk/manager.h 387741 
  /trunk/include/asterisk/stasis_channels.h 387741 
  /trunk/main/manager_channels.c 387741 
  /trunk/main/stasis_channels.c 387741 

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


Testing
-------

ChanSpy tests pass (once I remembered to install chan_console... ew)


Thanks,

Matt Jordan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130506/5142d4e8/attachment.htm>


More information about the asterisk-dev mailing list