Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem with rfc2833-pt and DTMF on bridged calls ever since. #1961

Open
mahald opened this issue Feb 10, 2023 · 3 comments
Open

Problem with rfc2833-pt and DTMF on bridged calls ever since. #1961

mahald opened this issue Feb 10, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@mahald
Copy link
Contributor

mahald commented Feb 10, 2023

Describe the bug

The DTMF transmission for bridged calls has been problematic for some time now, particularly when the setting "bypass_media_after_bridge=true" is used. This issue arises when the destination PBX utilizes a different PT value for RFC2833 than what is configured in the sofia profile.

To address this challenge, we have implemented a workaround that involves parsing the PT from the switch_r_sdp on the A leg and using a profile with the same value set as rfc2833-pt. For example, if the PT value is 101 on the A-Leg, we use the profile "sofia/outbound101", and if it is 103, we use "sofia/outbound103". This workaround has effectively resolved our DTMF transmission issues.

However, this solution requires extensive configuration and coding on our end, which some customers may not find favorable. Additionally, each profile must utilize a different source UDP Port, further complicating the matter.

It would be beneficial if there was a channel variable for "rtp_2833_payload" that takes precedence over the profile setting when it is set.

@mahald mahald added the bug Something isn't working label Feb 10, 2023
@briankwest
Copy link
Collaborator

Channel variables can't have dashes, it must be an _, and can you do a PR?

@mahald
Copy link
Contributor Author

mahald commented Aug 14, 2024

added pull request #2573 @briankwest can you please merge this? This Code is Live for > 1 year.

@mahald
Copy link
Contributor Author

mahald commented Nov 27, 2024

@briankwest can you please merge this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants