Skip to content

fix(sendDtmf): return success status and harden codec preference logic#9

Open
smoghe-bw wants to merge 1 commit into
mainfrom
remove-aggressive-dtmf-codec-setting
Open

fix(sendDtmf): return success status and harden codec preference logic#9
smoghe-bw wants to merge 1 commit into
mainfrom
remove-aggressive-dtmf-codec-setting

Conversation

@smoghe-bw

Copy link
Copy Markdown
Contributor

Make sendDtmf return true/false based on whether tones were queued on at least one stream, and only touch setCodecPreferences when the caller explicitly provides preferences, falling back safely on rejection across browser implementations.

Make sendDtmf return true/false based on whether tones were queued on
at least one stream, and only touch setCodecPreferences when the
caller explicitly provides preferences, falling back safely on
rejection across browser implementations.
@smoghe-bw smoghe-bw requested review from a team as code owners July 2, 2026 13:25
@bwappsec

bwappsec commented Jul 2, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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.

2 participants