Skip to content

Remove stale data track subscriber handles#2011

Open
ladvoc wants to merge 1 commit into
mainfrom
ladvoc/cleanup-old-data-track-handles
Open

Remove stale data track subscriber handles#2011
ladvoc wants to merge 1 commit into
mainfrom
ladvoc/cleanup-old-data-track-handles

Conversation

@ladvoc

@ladvoc ladvoc commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #2000

@changeset-bot

changeset-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 9df630a

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

@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size
dist/livekit-client.esm.mjs 101.44 KB (-0.05% 🔽)
dist/livekit-client.umd.js 110.43 KB (+0.07% 🔺)

@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: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@1egoman 1egoman 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.

Makes sense, good catch!

For my own understanding, can you outline a scenario where a packet would be received from an old subscribe handle after a full reconnect? Maybe something like an old packet was delayed when being sent over the network and takes longer than normal to be delivered?

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