[asterisk-dev] [Code Review] 3505: app_chanspy: Fix a bug where barge mode only works on the first connection when multiple sessions are spied on for a channel

[Joshua Colp]

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


So either I'm crazy or... I don't see a difference between the old and new code, except that the logic is in a separate function. Am I crazy?

- Joshua Colp


On May 2, 2014, 8:35 p.m., Jonathan Rose wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3505/
> -----------------------------------------------------------
> 
> (Updated May 2, 2014, 8:35 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-23381
>     https://issues.asterisk.org/jira/browse/ASTERISK-23381
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This is a cleanup of the patches posted by Robert Moss for ASTERISK-23381 - A stalled review of the patch exists here: https://reviewboard.asterisk.org/r/3331/diff/1/#index_header
> 
> This version is made to apply to Asterisk 11 since the bug was first reported there.
> 
> 
> Diffs
> -----
> 
>   /branches/11/apps/app_chanspy.c 413071 
> 
> Diff: https://reviewboard.asterisk.org/r/3505/diff/
> 
> 
> Testing
> -------
> 
> 1. SIP/A calls a spy application which spies on SIP/B with the DTMF option (4 for spy mode, 5 for whisper mode, 6 for barge)
> 2. SIP/B calls SIP/C
> 3. Check to see if audio is flowing from A to B and from A to C... should start as neither. (A always hears both of the others though)
>  * press DTMF 5
>   - now audio should flow from A to B, but from not A to C
>  * press DTMF 6
>   - now audio should flow from A to B and A to C
> 4. Hang up SIP/B
> 5. Have SIP/B call SIP/C again
> 6. The spy should reconnect automatically and be in barge mode
>   - audio should flow from A to B and A to C
>  * press DTMF 5
>   - audio should flow from A to B, but not from A to C
>  * press DTMF 4
>   - audio should not flow from A to either phone
> 
> 
> Prior to patch: Audio from A to C could not be heard in the first check
> After the patch: Audio from A to C can be heard in the first check
> 
> All other checks work as expected in both situations.
> 
> 
> Thanks,
> 
> Jonathan Rose
> 
>

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