[asterisk-dev] [Code Review] 3494: ARI: Add the ability to raise an arbitrary User Event from the Applications resource

Scott Griepentrog reviewboard at asterisk.org
Tue Apr 29 11:15:21 CDT 2014


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

Review request for Asterisk Developers.


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


Repository: Asterisk


Description
-------

This changes the implementation of UserEvent stasis messaging and adds additional features:

1) Existing channel_user_event stasis messaging replaced with new multi_object_blob in stasis_user.c
2) Multi object blob provides ability to signal blob + any number of channel, bridge, or endpoint snapshots
3) Changed existing Userevent() app to use new multi object blob - but still publishes to channel
4) Added ARI implementation of userevent as /ari/applications/{applicationName}/user/{eventName}
5) ARI generated userevent messages are published to application, and also to AMI if channel present


Diffs
-----

  /branches/12/rest-api/api-docs/events.json 413115 
  /branches/12/rest-api/api-docs/applications.json 413115 
  /branches/12/res/stasis/app.c 413115 
  /branches/12/res/res_stasis.c 413115 
  /branches/12/res/res_ari_applications.c 413115 
  /branches/12/res/ari/resource_applications.c 413115 
  /branches/12/res/ari/resource_applications.h 413115 
  /branches/12/res/ari/ari_model_validators.c 413115 
  /branches/12/res/ari/ari_model_validators.h 413115 
  /branches/12/main/stasis_user.c PRE-CREATION 
  /branches/12/main/stasis_channels.c 413115 
  /branches/12/main/manager_channels.c 413115 
  /branches/12/main/asterisk.c 413115 
  /branches/12/include/asterisk/stasis_user.h PRE-CREATION 
  /branches/12/include/asterisk/stasis_channels.h 413115 
  /branches/12/include/asterisk/stasis_app.h 413115 
  /branches/12/apps/app_userevent.c 413115 

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


Testing
-------

Tested with valgrind to insure no invalid references.  Some leaks found which have been attributed to other code to be fixed separately.


Thanks,

Scott Griepentrog

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140429/ccdceb3e/attachment.html>


More information about the asterisk-dev mailing list