Skip to content

Add Call.UseSFU to join an SFU without the coordinator - #4

Merged
tbarbugli merged 1 commit into
factory/3rttfrom
feature/use-sfu-directly
Oct 1, 2026
Merged

tbarbugli merged 1 commit into
factory/3rttfrom
feature/use-sfu-directly

Conversation

@tbarbugli

@tbarbugli tbarbugli commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Adds Call.UseSFU(models.Credentials), which points a call straight at an SFU with credentials the caller already holds, so Join skips the coordinator's join-call request.

This is for development, tests and benchmarks against a locally running SFU (no coordinator to hand out an SFU and token), or when the caller has already made the coordinator request itself and wants to pin an SFU. Production calls should keep getting both from the coordinator. It must be called before Join; reconnects and migrations keep using the same credentials, since there is no coordinator to ask for new ones.

Tested: GitHub Actions is blocked by org billing, so CI doesn't run on this PR; locally, make ci passes on the branch rebased onto factory/3rtt, and new TestUseSFUJoinsWithoutTheCoordinator joins a fake SFU with no coordinator configured. go build ./..., go vet ./..., gofumpt, go mod tidy and go test -race -count=1 ./... pass on the branch.

🤖 Generated with Claude Code

UseSFU takes SFU credentials obtained elsewhere, for example from a
coordinator JoinCall made by the caller or a locally running SFU, and
joins it directly.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@tbarbugli
tbarbugli force-pushed the feature/use-sfu-directly branch from 3111116 to d0fe8b6 Compare September 30, 2026 20:20
@tbarbugli
tbarbugli changed the base branch from main to factory/3rtt September 30, 2026 20:20
@tbarbugli
tbarbugli marked this pull request as ready for review September 30, 2026 21:36
@tbarbugli
tbarbugli merged commit 7b0ff0a into factory/3rtt Oct 1, 2026
2 of 3 checks passed
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.

1 participant