Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: participant updates #202

Merged
merged 1 commit into from
Sep 25, 2023
Merged

fix: participant updates #202

merged 1 commit into from
Sep 25, 2023

Conversation

theomonnom
Copy link
Member

@theomonnom theomonnom commented Sep 25, 2023

  • Handle the case a remote participant Sid can change
  • Don't create a new participant if the state of the new one is already disconnected (e.g)

[2023-09-25T23:23:31Z INFO livekit::room] participant update: [ParticipantInfo { sid: "PA_3DH5jgnRV8NA", identity: "client", state: Disconnected, tracks: [], metadata: "", joined_at: 1695684209, name: "", version: 3, permission: Some(ParticipantPermission { can_subscribe: true, can_publish: true, can_publish_data: true, can_publish_sources: [], hidden: false, recorder: false, can_update_metadata: false }), region: "sfo3", is_publisher: false }]

@theomonnom theomonnom merged commit 7da0966 into main Sep 25, 2023
14 checks passed
@theomonnom theomonnom deleted the theo/fix-participant-updates branch September 25, 2023 23:34
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