Skip to content

Stop audio publishing when audio session activation fails#808

Merged
hiroshihorie merged 3 commits intomainfrom
hiroshi/audio-session-activate-check
Mar 24, 2026
Merged

Stop audio publishing when audio session activation fails#808
hiroshihorie merged 3 commits intomainfrom
hiroshi/audio-session-activate-check

Conversation

@hiroshihorie
Copy link
Copy Markdown
Member

@hiroshihorie hiroshihorie commented Oct 8, 2025

More audio safety checks
How to test: try to publish when Siri or Phone call is active.

Base automatically changed from hiroshi/fix-renderer-adapter to main October 18, 2025 05:24
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 21, 2025

⚠️ This PR does not contain any files in the .changes directory.

@hiroshihorie hiroshihorie force-pushed the hiroshi/audio-session-activate-check branch 4 times, most recently from 2bbeb06 to 12dbd0c Compare March 17, 2026 08:25
@hiroshihorie hiroshihorie marked this pull request as ready for review March 17, 2026 08:37
@hiroshihorie hiroshihorie force-pushed the hiroshi/audio-session-activate-check branch from 12dbd0c to c20bcb0 Compare March 17, 2026 08:39
@hiroshihorie hiroshihorie requested a review from pblazej March 17, 2026 08:40
Comment thread Sources/LiveKit/Audio/AudioSessionEngineObserver.swift
Copy link
Copy Markdown
Contributor

@pblazej pblazej left a comment

Choose a reason for hiding this comment

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

I haven't encountered a crash on main, but that's definitely a step in the right direction 💯

Left 1 comment worth fixing.

Copy link
Copy Markdown
Contributor

@pblazej pblazej left a comment

Choose a reason for hiding this comment

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

LGTM

@hiroshihorie hiroshihorie merged commit e0f6389 into main Mar 24, 2026
28 checks passed
@hiroshihorie hiroshihorie deleted the hiroshi/audio-session-activate-check branch March 24, 2026 08:43
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