[bug] `AudioPublishOptions.red` is inverted: the default `red: true` sends `disableRed: true`, disabling RED for every audio publish
メンテナーはふだん 2 日以内に返信
評価
この issue はまだ評価されていません。
説明
Describe the bug
AudioPublishOptions.red (default true, documented as "red (Redundant Audio Data)") is passed into AddTrackRequest.disableRed without negation, so every audio publish with default options disables RED:
// lib/src/participant/local.dart
disableDtx: !publishOptions.dtx, // negated — correct
disableRed: room.e2eeManager != null ? true : publishOptions.red ?? true, // not negated
The JS SDK negates it (src/room/participant/LocalParticipant.ts):
disableRed: this.isE2EEEnabled || !(opts.red ?? true),
Evidence:
- Introduced by 6323ff7 (#453, 2024-01-23); the inverted line is that commit's only change to
lib/src/participant/local.dart. Before it the field was never set, and the protobuf default (false) left RED enabled — so RED has been off for Flutter publishers since v2.0.0. - The line is identical in 2.3.0–2.13.0, the v2.13.2 tag, and current main (checked 2026-10-30).
- The docs recommend RED enabled and document
red: falseas the opt-out: "LiveKit recommends enabling this feature because audio glitches are so distracting that the tradeoff is almost always worth it" — https://docs.livekit.io/transport/media/advanced/ - The E2EE force-disable branch matches the JS SDK; only the non-E2EE branch is inverted.
To Reproduce
Publish a microphone track with default options: the AddTrackRequest carries disableRed: true. Publish with red: false: it carries disableRed: false — inverted.
Expected behavior
red: true (the default) → disableRed: false. Proposed fix:
disableRed: room.e2eeManager != null || !(publishOptions.red ?? true),
Release-note caveat: apps that noticed and set red: false to force RED on will flip behavior when this is fixed.
Platform information
- Flutter version: 3.47.6
- Plugin version: 2.13.0
- Flutter target OS: Web — but the field is set in shared Dart, so all platforms are affected
- 主要言語
- Dart
- スター
- 421
- フォーク
- 260
- 平均マージ
- 2日 8時間
- マージ済み PR（30日）
- 25
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
livekit/client-sdk-flutter のほかの issue
-
`jwtPayload.roomConfiguration` returns null fields for tokens from LiveKit server SDKs対応中かも @RaphaelFakhri が 17 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
livekit/client-sdk-flutter#1243 ·
メンテナーはふだん 2 日以内に返信
-
難易度 3/5 1〜2日 初心者へのやさしさ 64/100
livekit/client-sdk-flutter#1274 ·
メンテナーはふだん 2 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 38/100
livekit/client-sdk-flutter#1265 ·
メンテナーはふだん 2 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
livekit/client-sdk-flutter#1227 ·
メンテナーはふだん 2 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
livekit/client-sdk-flutter#1221 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
livekit/client-sdk-flutter の issue をすべて見る
似ている issue
-
check-relative-paths reports links to existing directories as missing対応中かも @reidbaker-agent が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
google/skills_lint.dart#202 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
InlitX/GymMane#311 · リアクション 1 件 ·
メンテナーはふだん 4 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
zjs81/meshcore-open#634 ·
メンテナーはふだん 1 日以内に返信
-
Flutter: _configureLogging passes a log level to rac_configure_logging, which expects an environmentオープンbinding:flutter bug p2
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
RunanywhereAI/runanywhere-sdks#1246 ·
メンテナーはふだん 1 日以内に返信