Skip to content

Option for webrtc WARP - #1342

Merged
cnderrauber merged 7 commits into
mainfrom
warp_option
Aug 20, 2026
Merged

Option for webrtc WARP#1342
cnderrauber merged 7 commits into
mainfrom
warp_option

Conversation

@cnderrauber

Copy link
Copy Markdown
Contributor

Before you submit your PR

Make sure the following is true before submitting your PR:

  • I have read the contributing guidelines and validated that this PR will be accepted.
  • I have read and followed the principles regarding breaking changes, testing, and code quality.

PR description

Option for webrtc WARP, turned on by default in livekit runtime. The SFU will control it by sdp negotiation and ice attr.

Breaking changes

MSRV

If the PR modifies the crate's MSRV (Minimum Supported Rust Version), document it here.

Testing

Ideally, unit test the code you add, but ensure you're not repeating existing test cases. Use as many already written scaffolding, utilities as possible; write your own, when needed. If external services, APIs, tokens are required (e.g., running an LK server instance), provide the necessary information. Make sure your tests perform useful, context-aware assertions and do not simply emulate "happy paths".

Async

We want the project to be runtime-agnostic, so please reuse what's already in livekit-runtime and feel free to add anything missing. It's ok to use Tokio directly, when writing unit tests, if necessary. When testing, do not use artificial delays for the state to "catch up"; instead, respect the event flow and subscribe properly using channels or other mechanisms.

Default on, controlled by sfu feature (sdp, ice attr negotiation)
@github-actions

github-actions Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Changeset ✓

This PR includes a changeset covering all affected packages:

Package Bump
libwebrtc minor
livekit minor
livekit-ffi patch
webrtc-sys patch

devin-ai-integration[bot]

This comment was marked as resolved.

cnderrauber and others added 4 commits August 20, 2026 11:51
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
devin-ai-integration[bot]

This comment was marked as resolved.

cnderrauber and others added 2 commits August 20, 2026 13:22
Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>

@cloudwebrtc cloudwebrtc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm!

@cnderrauber
cnderrauber merged commit db3469b into main Aug 20, 2026
25 checks passed
@cnderrauber
cnderrauber deleted the warp_option branch August 20, 2026 06:55
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