-
Notifications
You must be signed in to change notification settings - Fork 21
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
Bug: Complex Setup with Streams and Group metadata causes forked groups #416
Comments
Logs with extra logging from Nick's libxmtp branch: What seems notable to me is that it looks like the commit for updating the group name is being processed by both clients, however we start seeing logs
|
It feels like we are getting somewhere. Maybe it would be helpful to compare the logs with a run through the same actions that doesn't fork the group state. Then we can see what's different. Some It would also be helpful to start logging hashes of the envelope payloads all over the place so we can see exactly which messages are failing to process. That way when a message fails to decrypt we can tie it back to when it was sent and figure out what intent it was related to (adding the member, changing the name) and whether or not it was a retry. |
One other thing I could see being helpful to log is any transaction rollbacks. Want to make super sure that messages which appear to have been processed successfully aren't rolling back at the last minute silently somewhere. Maybe we just throw a line somewhere in the Storage |
New error log, with envelope hashes and messages on transaction commits and rollbacks, replaced inboxIDs with CREATOR and USER2 for readability: logs
|
Describe the bug
When setting streamAllMessages and updating group metadata fairly quickly after group creation
Group syncing stops working
Expected behavior
No response
Steps to reproduce the bug
No response
The text was updated successfully, but these errors were encountered: