Skip to content

Use subscriber connection as primary#41

Merged
davidzhao merged 3 commits intomainfrom
sub-as-primary
Sep 12, 2021
Merged

Use subscriber connection as primary#41
davidzhao merged 3 commits intomainfrom
sub-as-primary

Conversation

@davidzhao
Copy link
Copy Markdown
Member

Implements client protocol version 3 - subscriber primary peer connections. Designed to reduce publisher connection load when the client is in read-only mode.

  • rely on subscriber connection as the primary PeerConnection
  • backwards compatible with previous server versions (and uses publisher as primary)
  • receive data messages on subscriber PC

Copy link
Copy Markdown
Member

@frostbyte73 frostbyte73 left a comment

Choose a reason for hiding this comment

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

lgtm

@davidzhao davidzhao merged commit 6865dc4 into main Sep 12, 2021
@davidzhao davidzhao deleted the sub-as-primary branch September 12, 2021 20:29
coryetzkorn pushed a commit to coryetzkorn/client-sdk-js that referenced this pull request Nov 5, 2025
afgarcia86 pushed a commit to afgarcia86/client-sdk-js that referenced this pull request Mar 19, 2026
* rename attributes to props for consistency

* fix MediaTrack attributes not merged into props

* use correct mergeProps and fix ts error

* remove box shadow for pressed state

* add pointer cursor to ui buttons

* set participant name also as a data attribute
add some TODOs
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