freeswitch icon indicating copy to clipboard operation
freeswitch copied to clipboard

mod_conference: member hold api doesn't work. The member can still hear the conference

Open TelephonyMan opened this issue 5 years ago • 4 comments

When a conference member is put on hold, the member should be both deaf and mute. Instead, the member can hear the other participants talking. I looked at the code for conference_api_sub_hold(), it only puts the call on mute.

Member "hold" followed by member "deaf" does give the desired effect, but that is two different API calls.

Is this by design or a defect?

TelephonyMan avatar Aug 14 '20 06:08 TelephonyMan

it sets MFLAG_HOLD which should make the user not hear anything. This was backported from an internal branch where we made the feature for our own needs, if its not working its missing in a condition in conference loop somewhere. Look at the places where we check for the deaf flag that dont also check for hold flag.

mjerris avatar Aug 15 '20 22:08 mjerris

was it fixed ? when I put the call on hold I still hear ringback when doing bgdial

benoitc avatar Mar 10 '25 19:03 benoitc

Ah, I forgot all about this. Yes, I did fix it in my local copy and moved on. Since no one else confirmed the bug, it simply slipped my mind. I can create a PR and suggest my fix.

TelephonyMan avatar Mar 11 '25 10:03 TelephonyMan

Here is the PR: https://github.com/signalwire/freeswitch/pull/2782

Hope it will be approved. I have used this for 4 years in production at various call centers.

TelephonyMan avatar Mar 11 '25 11:03 TelephonyMan