[asterisk-dev] [Code Review] 3025: ARI: Implement device state API

Kevin Harwell reviewboard at asterisk.org
Sat Nov 23 11:35:03 CST 2013


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

(Updated Nov. 23, 2013, 11:38 a.m.)


Status
------

This change has been marked as submitted.


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


Changes
-------

Committed in revision 403134


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


Repository: Asterisk


Description
-------

Created a data model and implemented functionality for an ARI device state resource.  The following operations have been added that allow a user to manipulate an ARI controlled device:

PUT    /device-states/{deviceName}&{deviceState} - Create/Change the state of an ARI controlled device
GET    /device-states/{deviceName} - Retrieve the current state of a device
DELETE /device-states/{deviceName} - Destroy a device-state controlled by ARI

The ARI controlled device must begin with 'Stasis:'.  An example controlled device name would be Stasis:Example.

A 'DeviceStateChanged' event has also been added so that an application can subscribe and receive device change events.  Any device state, ARI controlled or not, can be subscribed to.

While adding the event, the underlying subscription control mechanism was refactored so that all current and future resource subscriptions would be the same.  Each event resource must now register itself in order to be able to properly handle [un]subscribes.


Diffs
-----

  branches/12/rest-api/resources.json 403013 
  branches/12/rest-api/api-docs/events.json 403013 
  branches/12/rest-api/api-docs/deviceStates.json PRE-CREATION 
  branches/12/rest-api/api-docs/applications.json 403013 
  branches/12/rest-api-templates/ari.make.mustache 403013 
  branches/12/res/stasis/app.c 403013 
  branches/12/res/stasis/app.h 403013 
  branches/12/res/res_stasis_device_state.exports.in PRE-CREATION 
  branches/12/res/res_stasis_device_state.c PRE-CREATION 
  branches/12/res/res_stasis.c 403013 
  branches/12/res/res_ari_device_states.c PRE-CREATION 
  branches/12/res/ari/resource_device_states.c PRE-CREATION 
  branches/12/res/ari/resource_device_states.h PRE-CREATION 
  branches/12/res/ari/resource_applications.h 403013 
  branches/12/res/ari/ari_model_validators.c 403013 
  branches/12/res/ari/ari_model_validators.h 403013 
  branches/12/res/ari.make 403013 
  branches/12/main/devicestate.c 403013 
  branches/12/include/asterisk/stasis_app_device_state.h PRE-CREATION 
  branches/12/include/asterisk/stasis_app.h 403013 
  branches/12/include/asterisk/devicestate.h 403013 

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


Testing
-------

Some basic testing done using the swagger framework and a websocket connection.  Testing adding, changing, and removing device state with regards to an ARI controlled device were successful.  Also tested [un]subscribing to the events from device with success.  Planning on writing some testsuite test cases as well.


Thanks,

Kevin Harwell

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


More information about the asterisk-dev mailing list