[asterisk-dev] [Code Review] 2987: ARI: Don't leak information about implementation details

[opticron]

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

Review request for Asterisk Developers.


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


Repository: Asterisk


Description
-------

This change prevents channels used as implementation details from leaking out to ARI. It does this by preventing creation of JSON blobs of channel snapshots created from those channels and sanitizing JSON blobs of bridge snapshots as they are created.

This results in NULL inputs to ast_json_pack calls which generate unhelpful error messages, so that has been dealt with as well.

This also corrects a bug I noticed while investigating the issue where BridgeCreated events would not be created.


Diffs
-----

  branches/12/res/stasis/app.c 402347 
  branches/12/main/stasis_channels.c 402347 
  branches/12/main/stasis_bridges.c 402347 
  branches/12/main/json.c 402347 

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


Testing
-------

Manual testing with bridges and channels.


Thanks,

opticron

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