----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3494/#review11809 -----------------------------------------------------------
/branches/12/rest-api/api-docs/applications.json <https://reviewboard.asterisk.org/r/3494/#comment21653> What was the thought process behind adding this API endpoint to the applications resource? Would it not make more sense to generate custom user events from the events resource? /branches/12/rest-api/api-docs/applications.json <https://reviewboard.asterisk.org/r/3494/#comment21654> Does this support passing in json to be returned through a web socket event or does the consumer of the web socket event have to parse the string properly? - Samuel Galarneau On April 29, 2014, 4:15 p.m., Scott Griepentrog wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3494/ > ----------------------------------------------------------- > > (Updated April 29, 2014, 4:15 p.m.) > > > 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 > >
-- _____________________________________________________________________ -- 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