8000 Handle errors in receive methods · Issue #258 · composablesys/collabs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Handle errors in receive methods #258

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.

Sign up for GitHub

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

Jump to bottom
Open
mweidner037 opened this issue Jul 15, 2023 · 0 comments
Open

Handle errors in receive methods #258

mweidner037 opened this issue Jul 15, 2023 · 0 comments
Labels
package:core core package package:crdts crdts package

Comments

@mweidner037
Copy link
Collaborator

Due to custom Collab implementations and #257, we should occasional errors in Collab.receive methods.

Currently, these will propagate up the stack and the leave the app in an inconsistent and possibly stuck state. This behavior is okay for local operations (crash the app without affecting any collaborators), but probably not when receiving remote messages - a single bad message could make a doc unusable.

It would be nice to find a better way to handle these errors. In particular, we could ensure that all parts of a transaction that don't cause an error go through normally, including the vector clock update and messages for other CRDTs.
@mweidner037 mweidner037 added package:core core package package:crdts crdts package labels Jul 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
package:core core package package:crdts crdts package
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mweidner037
0