Skip to content

Mark data track unpublished after republish fail - #2170

Open
1egoman wants to merge 1 commit into
mainfrom
datatrack-republish-rejected
Open

1egoman wants to merge 1 commit into
mainfrom
datatrack-republish-rejected

Conversation

@1egoman

@1egoman 1egoman commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

Port of livekit/rust-sdks#1537.

When the SFU rejects a republish after a full reconnect, set the local data track's descriptor to unpublished instead of leaving it in republishing, and log the rejection error. Adds a test for a rejected republish, ported from test_republish_rejected.

Note: in JS, receivedSfuPublishResponse already removes the descriptor from the map before handling the response. A track whose republish was rejected was therefore already reported as unpublished through isPublished() and tryPush, and the new test also passes without the change. The change keeps the state handling the same as the Rust SDK.

🤖 Generated with Claude Code

Port of livekit/rust-sdks#1537.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@changeset-bot

changeset-bot Bot commented Oct 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 1127368

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
livekit-client Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@devin-ai-integration devin-ai-integration Bot 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.

🔍 Devin Review: 1 flag

Not posted on this PR by your GitHub settings — view it in Devin Review. (Configure)

Devin Review

@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size
dist/livekit-client.esm.mjs 115.81 KB (-0.03% 🔽)
dist/livekit-client.umd.js 125.15 KB (-0.03% 🔽)
dist/core/core.mjs 78.55 KB (0%)

@1egoman

1egoman commented Oct 21, 2026

Copy link
Copy Markdown
Contributor Author

Going to wait for @ladvoc to take a look before I merge this, since he wrote the original rust sdks PR that this ports over.

This branch has not been deployed

No deployments
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