[bug] `AudioPublishOptions.red` is inverted: the default `red: true` sends `disableRed: true`, disabling RED for every audio publish

未关闭
#1,271 0 条评论 0 个 reaction 已指派 1 人 在 GitHub 查看

维护者通常 2 天内回复

@hiroshihorie 已经在做这个了。

开始于 2026年10月30日。

  • #1272 来自 @hiroshihorie —— 未关闭

评估

这个 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: false as 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 天 5 小时
30 天内合并 PR
20

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

livekit/client-sdk-flutter 的其他 Issue

查看 livekit/client-sdk-flutter 的全部 Issue

相似的 Issue

更多 Dart Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。