[asterisk-dev] [Code Review] 2862: ARI: Add subscription support

[David Lee]

> On Sept. 26, 2013, 3:33 p.m., Matt Jordan wrote:
> > /branches/12/main/endpoints.c, lines 92-103
> > <https://reviewboard.asterisk.org/r/2862/diff/1/?file=46016#file46016line92>
> >
> >     See Richard's template for comparison callbacks:
> >     
> >     https://wiki.asterisk.org/wiki/pages/viewpage.action?pageId=25919686

I don't see a template for comparison callbacks; just hash and sort.
Am I missing it?


> On Sept. 26, 2013, 3:33 p.m., Matt Jordan wrote:
> > /branches/12/res/ari/resource_applications.c, lines 87-95
> > <https://reviewboard.asterisk.org/r/2862/diff/1/?file=46022#file46022line87>
> >
> >     Do you want to validate args->application_name here as well?
> >     
> >     if (ast_strlen_zero(args->application_name)) {
> >         // send 400
> >     }
> >     
> >     That way stasis_app_subscribe doesn't have to be quite as paranoid.

I'll add the check, but keep the paranoia :-)


- David


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


On Sept. 17, 2013, 11:08 a.m., David Lee wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2862/
> -----------------------------------------------------------
> 
> (Updated Sept. 17, 2013, 11:08 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-22451
>     https://issues.asterisk.org/jira/browse/ASTERISK-22451
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This patch adds an /applications API to ARI, allowing explicit management of
> Stasis applications.
> 
>  * GET /applications - list current applications
>  * GET /applications/{applicationName} - get details of a specific application
>  * POST /applications/{applicationName}/subscription - explicitly subscribe to
>    a channel, bridge or endpoint
>  * DELETE /applications/{applicationName}/subscription - explicitly unsubscribe
>    from a channel, bridge or endpoint
> 
> Subscriptions work by a reference counting mechanism: if you subscript to an
> event source X number of times, you must unsubscribe X number of times to stop
> receiveing events for that event source.
> 
> 
> Diffs
> -----
> 
>   /branches/12/include/asterisk/endpoints.h 399236 
>   /branches/12/include/asterisk/stasis_app.h 399236 
>   /branches/12/main/asterisk.c 399236 
>   /branches/12/main/endpoints.c 399236 
>   /branches/12/main/json.c 399236 
>   /branches/12/res/ari.make 399236 
>   /branches/12/res/ari/ari_model_validators.h 399236 
>   /branches/12/res/ari/ari_model_validators.c 399236 
>   /branches/12/res/ari/resource_applications.h PRE-CREATION 
>   /branches/12/res/ari/resource_applications.c PRE-CREATION 
>   /branches/12/res/ari/resource_endpoints.h 399236 
>   /branches/12/res/res_ari_applications.c PRE-CREATION 
>   /branches/12/res/res_ari_model.c 399236 
>   /branches/12/res/res_stasis.c 399236 
>   /branches/12/res/stasis/app.h 399236 
>   /branches/12/res/stasis/app.c 399236 
>   /branches/12/rest-api-templates/ari_model_validators.h.mustache 399236 
>   /branches/12/rest-api/api-docs/applications.json PRE-CREATION 
>   /branches/12/rest-api/api-docs/endpoints.json 399236 
>   /branches/12/rest-api/api-docs/events.json 399236 
>   /branches/12/rest-api/resources.json 399236 
> 
> Diff: https://reviewboard.asterisk.org/r/2862/diff/
> 
> 
> Testing
> -------
> 
> Testsuite tests (https://reviewboard.asterisk.org/r/2863/)
> 
> I hit a roadblock testing the endpoint subscription code, so that was tested
> manually using wscat and Swagger-UI.
> 
> 
> Thanks,
> 
> David Lee
> 
>

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