Re: [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
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated May 9, 2014, 11:10 a.m.) Status -- This change has been marked as submitted. Review request for Asterisk Developers. Changes --- Committed in revision 413551 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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated May 8, 2014, 2:31 p.m.) Review request for Asterisk Developers. Changes --- Nah, you weren't crazy. I thought I had determined the change to be unnecessary for some reason, but then I realized the cause of the bug was slightly different than how I thought it was when I first reproduced it. Instead I'm getting rid of the first call to attach the audiohook and keeping the one inside the loop. If the channel isn't bridged with anything when the attach_barge function is called, then it will keep trying until it can attach one. 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 (updated) - /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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/#review11860 --- Ship it! Ship It! - Joshua Colp On May 8, 2014, 7:31 p.m., Jonathan Rose wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated May 8, 2014, 7:31 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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
On May 1, 2014, 1:37 p.m., Mark Michelson wrote: Ship It! It's too bad Reviewboard doesn't have a method for cancelling a ship it since this applied to an older version of the patch. - Jonathan --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/#review11801 --- On May 2, 2014, 3: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, 3: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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
--- 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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
On May 1, 2014, 1:14 p.m., Joshua Colp wrote: /branches/11/apps/app_chanspy.c, lines 664-668 https://reviewboard.asterisk.org/r/3505/diff/1/?file=58139#file58139line664 Can you explain why the audiohook would not already be attached by the previous code here? Joshua Colp wrote: Or to be more pedantic: by the code that was invoked when the function was entered. After some experimenting, it turns out there is no reason for the audiohook not to be attached by the previous call. At least not after the changes that have been made. I'll be posting a new version shortly. - Jonathan --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/#review11799 --- On April 29, 2014, 5:07 p.m., Jonathan Rose wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated April 29, 2014, 5:07 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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/#review11818 --- /branches/11/apps/app_chanspy.c https://reviewboard.asterisk.org/r/3505/#comment21679 spyee_bridge_autochan doesn't propogate back to the caller in this function, so it will never be free'd. This is fixed in the next version of the patch. - Jonathan Rose On April 29, 2014, 5:07 p.m., Jonathan Rose wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated April 29, 2014, 5:07 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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated May 2, 2014, 3:35 p.m.) Review request for Asterisk Developers. Changes --- Eliminates second audiohook attachment function call and deals with a bug I noticed from the original patch where pointers were being used improperly. 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 (updated) - /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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
On May 1, 2014, 6:14 p.m., Joshua Colp wrote: /branches/11/apps/app_chanspy.c, lines 664-668 https://reviewboard.asterisk.org/r/3505/diff/1/?file=58139#file58139line664 Can you explain why the audiohook would not already be attached by the previous code here? Or to be more pedantic: by the code that was invoked when the function was entered. - Joshua --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/#review11799 --- On April 29, 2014, 10:07 p.m., Jonathan Rose wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated April 29, 2014, 10:07 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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
Re: [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
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/#review11801 --- Ship it! Ship It! - Mark Michelson On April 29, 2014, 10:07 p.m., Jonathan Rose wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- (Updated April 29, 2014, 10:07 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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
[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
--- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3505/ --- 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 -- _ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev