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(JitsiConference): sync up local tracks #2469

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Mar 12, 2024

  1. fix(JitsiConference): sync up local tracks

    It can happen that a new session is started while JingleSessionPC
    operation is in progress.
    
    Queue up the track operation on the new session to make sure it
    catches up with the correct tracks state.
    
    Example:
    1. JitsiConference.replaceTrack(camera1, camera2);
    2. At the time there's only JVB session, it's replaceTrack will
       take 100ms.
    3. 50 ms later - a P2P session is started, it will use camera1
       as "local tracks" to start the session, because the
       replacement has not finished yet and this.rtc.localTracks
       still contains camera1.
    4. Another 50ms later, JVB session finishes the replaceTrack
       and here we see that there's a new P2P session. Schedule
       a catch-up replaceTrack operation and make it part of the
       original addTrack operation chain.
    paweldomas committed Mar 12, 2024
    Configuration menu
    Copy the full SHA
    e97645e View commit details
    Browse the repository at this point in the history