[asterisk-dev] [Code Review] 3810: res_hep_rtcp: Add module that sends RTCP information to a Homer Server

[opticron]

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


The Asterisk side of this looks good. Could you add the review(s) for the test(s) to this review's dependencies?

- opticron


On July 16, 2014, 5:37 p.m., Matt Jordan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3810/
> -----------------------------------------------------------
> 
> (Updated July 16, 2014, 5:37 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This patch adds a new module to Asterisk, res_hep_rtcp. The module subscribes to Stasis and receives RTCP information back from the message bus, which it encodes into HEPv3 packets and sends to the res_hep module for transmission.
> 
> Using this, someone with a Homer server can get live call quality monitoring for all channels in their Asterisk 12+ systems.
> 
> 
> Diffs
> -----
> 
>   /branches/12/res/res_rtp_asterisk.c 418783 
>   /branches/12/res/res_hep_rtcp.c PRE-CREATION 
>   /branches/12/CHANGES 418783 
> 
> Diff: https://reviewboard.asterisk.org/r/3810/diff/
> 
> 
> Testing
> -------
> 
> Some manual testing has be done, and automated tests have started being written in the Asterisk Test Suite. Note that they need to be completed before this is submitted.
> 
> As a side note, Alexander actually demo'd this at Kamailio World - you can see it on the 'dangerous demos' here - http://www.youtube.com/watch?v=ykBdOTCCSHs
> 
> 
> Thanks,
> 
> Matt Jordan
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140718/7f597e49/attachment.html>