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(sync): Disable browser broadcast #6479

Merged
merged 1 commit into from
Oct 1, 2024
Merged

Commits on Sep 30, 2024

  1. fix(sync): Disable browser broadcast

    Ensure the browser broadcast does not lead to out of sync docs.
    
    If a tab fails to send a step over the network
    it may still distributes it to a different tab / window
    via the browser broadcast.
    
    That other window will include the step in the yjs history
    without forwarding it to other parties.
    
    From that point on all changes in the second tab
    will depend on the step that other clients did not receive.
    
    Therefore these changes will not be applied at other parties.
    
    To prevent this scenario disable browser broadcast completely.
    
    Signed-off-by: Max <[email protected]>
    max-nextcloud committed Sep 30, 2024
    Configuration menu
    Copy the full SHA
    527fc31 View commit details
    Browse the repository at this point in the history