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

TT#14008 DTX buffer PT change: take supp PTs into account #1509

Closed
wants to merge 6 commits into from

Conversation

szcom
Copy link
Contributor

@szcom szcom commented Jun 30, 2022

The PT tracker doesn't distinguish between audio/media types and
supplemental types, so in order not to break DTMF handling we must take
all combinations of primary (audio/media) types and supplemental types
as both input types and handler types into account.

Fix-up for 74075f6
Fix-up for I57e1278e4fad157083d9526d4829f2940581687f

closes #1508
possibly also #1504

Change-Id: If7b242def2d35fbed14b11d204ea328b8bfe5d79

rfuchs and others added 6 commits June 30, 2022 10:53
The PT tracker doesn't distinguish between audio/media types and
supplemental types, so in order not to break DTMF handling we must take
all combinations of primary (audio/media) types and supplemental types
as both input types and handler types into account.

Fix-up for 74075f6
Fix-up for I57e1278e4fad157083d9526d4829f2940581687f

closes sipwise#1508
possibly also sipwise#1504

Change-Id: If7b242def2d35fbed14b11d204ea328b8bfe5d79
@rfuchs
Copy link
Member

rfuchs commented Mar 30, 2023

AFAICS there are all merged/fixed

@rfuchs rfuchs closed this Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

some DTMF events are not relayed in transcoding scenario
2 participants