[asterisk-dev] [Code Review] 2490: stasis: Swap out network_change events for network_change stasis messages sent to a new 'system' topic.

jrose reviewboard at asterisk.org
Thu May 2 10:55:57 CDT 2013


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

(Updated May 2, 2013, 3:55 p.m.)


Review request for Asterisk Developers, David Lee, kmoore, and Matt Jordan.


Changes
-------

Address Matt's review.

AST_EVENT_DEVICE_STATE_CHANGE remains defined due to res_corosync.


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


Repository: Asterisk


Description
-------

A continuation of the effort started with https://reviewboard.asterisk.org/r/2481

This one replaces the NETWORK_CHANGE events. I added the system topic and it is stored in/is registed by asterisk.c at startup.
It's worth noting that NETWORK_CHANGE events had no data associated with them before. Since generating stasis messages ao2_refs the
data going in, some ao2_object has to be included, so I went ahead and gave it a blank JSON object wrapped in a json_payload similar
to how I handled the ACL change messages.

Part 2/3


Diffs (updated)
-----

  /trunk/res/res_stun_monitor.c 387038 
  /trunk/main/event.c 387038 
  /trunk/main/asterisk.c 387038 
  /trunk/include/asterisk.h 387038 
  /trunk/include/asterisk/event_defs.h 387038 
  /trunk/channels/chan_sip.c 387038 
  /trunk/channels/chan_iax2.c 387038 

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


Testing
-------

Since the only thing that generates these messages under normal conditions is res_stun_monitor and I don't really know how to or particularly want to set up a stun server, I simply used a test function that would generate the messages in the same way as they are when the STUN monitor would issue them. I made sure that each of the consumers (which are just chan_sip and chan_iax2) reacted to the messages as anticipated and they did.


Thanks,

jrose

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


More information about the asterisk-dev mailing list