Skip to content

Conversation

@piitaya
Copy link
Member

@piitaya piitaya commented Jun 12, 2025

Proposed change

Don't add audio track if webrtc player is muted.
Need more test before merging.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

hlsStreams.hasAudio &&
!webRtcStreams.hasAudio
!webRtcStreams.hasAudio &&
!this.muted
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That means we need to include muted in the memoized function variables?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bramkragten Hi,
There is an estimate time this fix will be published?

Thank you!

@piitaya piitaya force-pushed the webrtc_audio_muted_no_track branch from 1228f17 to 7a06aa5 Compare October 14, 2025 10:09
@piitaya piitaya requested a review from bramkragten October 14, 2025 10:14
@piitaya piitaya marked this pull request as ready for review October 14, 2025 10:14
@bramkragten bramkragten enabled auto-merge (squash) October 14, 2025 10:20
@bramkragten bramkragten merged commit 938128d into dev Oct 14, 2025
11 checks passed
@bramkragten bramkragten deleted the webrtc_audio_muted_no_track branch October 14, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

WebRTC cameras open an audio stream and hold it

4 participants