[asterisk-dev] [Code Review] 2507: Add support for indexing installed sounds and for Stasis-HTTP sounds queries

svnbot reviewboard at asterisk.org
Mon Jun 24 08:49:27 CDT 2013


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

(Updated June 24, 2013, 8:49 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and David Lee.


Changes
-------

Committed in revision 392700


Bugs: ASTERISK-21584 and ASTERISK-21585
    https://issues.asterisk.org/jira/browse/ASTERISK-21584
    https://issues.asterisk.org/jira/browse/ASTERISK-21585


Repository: Asterisk


Description
-------

This adds support for stasis/sounds and stasis/sounds/{ID} queries via the Stasis-HTTP interface.

The following changes have been made to accomplish this:
* A modular indexer was created for local media.
* A new function to get an ast_format associated with a file extension was added.  
* Modifications were made to the built-in HTTP server so that URI decoding could be deferred to the URI handler when necessary.
* The Stasis-HTTP sounds JSON documentation was modified to handle cases where multiple languages are installed in different formats.
* Register and Unregister events for formats were added to the system topic.


Diffs
-----

  trunk/include/asterisk/_private.h 391293 
  trunk/include/asterisk/file.h 391293 
  trunk/include/asterisk/format.h 391293 
  trunk/include/asterisk/http.h 391293 
  trunk/include/asterisk/media_index.h PRE-CREATION 
  trunk/include/asterisk/sounds_index.h PRE-CREATION 
  trunk/main/asterisk.c 391293 
  trunk/main/file.c 391293 
  trunk/main/http.c 391293 
  trunk/main/media_index.c PRE-CREATION 
  trunk/main/sounds_index.c PRE-CREATION 
  trunk/res/res_stasis_http.c 391293 
  trunk/res/stasis_http/resource_sounds.c 391293 
  trunk/rest-api/api-docs/sounds.json 391293 

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


Testing
-------

Tested using a browser and HTTP GET requests using the Stasis-HTTP API.  Currently contemplating how to create automated tests.


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130624/8546bd62/attachment.htm>


More information about the asterisk-dev mailing list