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

[opticron]

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



/branches/12/include/asterisk/endpoints.h
<https://reviewboard.asterisk.org/r/2862/#comment18991>

    By convention, this goes in include/asterisk/_private.h



/branches/12/include/asterisk/endpoints.h
<https://reviewboard.asterisk.org/r/2862/#comment18992>

    s/endpoing/endpoint/



/branches/12/include/asterisk/stasis_app.h
<https://reviewboard.asterisk.org/r/2862/#comment18993>

    Documentation for the json parameter is missing. Also, add a blank line between the params block and return block in the documentation.



/branches/12/include/asterisk/stasis_app.h
<https://reviewboard.asterisk.org/r/2862/#comment18994>

    Idem.



/branches/12/main/endpoints.c
<https://reviewboard.asterisk.org/r/2862/#comment18995>

    Use OBJ_SEARCH_MASK.



/branches/12/main/endpoints.c
<https://reviewboard.asterisk.org/r/2862/#comment19000>

    OBJ_SEARCH_KEY instead of OBJ_KEY. There are several more of these throughout the changes.



/branches/12/main/endpoints.c
<https://reviewboard.asterisk.org/r/2862/#comment19001>

    OBJ_SEARCH_OBJECT



/branches/12/res/ari/resource_applications.c
<https://reviewboard.asterisk.org/r/2862/#comment18998>

    This should check that the JSON array length corresponds with the number of apps in the container in case the ast_json_array_append() or stasis_app_to_json() calls failed in the callback unless we want to ignore this error condition.


- opticron


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/20130926/a14618b1/attachment-0001.html>