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

Track and reset KX with long offline users #116

Merged
merged 14 commits into from Feb 14, 2023

Conversation

miki-totefu
Copy link
Collaborator

This PR makes tracking and performing KX reset actions with users from which messages have not been received for some time easier.

  • Adds tracking for the last encrypt/decrypt time of ratchet messages.
  • Adds a function to reset the kx with all users from which no message has been received in X time
  • Automatically resets all KXs if the local client has been offline for longer than the server's message retention policy
  • Adds /rresetold and /ls userslastmsgtime commands to CLI
  • Adds a page to list users last msg time in GUI

miki added 9 commits January 27, 2022 14:57
This fixes a bug that prevented client shutdown if the client failed to
completely initialize during address book loading.
This ensures files that are unlikely to be KX files are not loaded
during startup.
This is added to the /addressbook command, as an optional argument to
display the addressbook entry of a particular user.
This resets all clients with ratchets that have not received messages in
a given age.
This allows showing the last time a message was received from all users.
This adds a check after connecting to the server to verify if the local
client has been offline for longer than the server's message retention
policy. If true, then the client sends a reset KX command to all known
users.

This is needed to prevent a situation where, after being offline for too
long, all of the client's ratchets are broken due to lost messages.
Instead of using a fixed 7-day interval, use the server expiration
policy.
@miki-totefu miki-totefu force-pushed the autorekx branch 7 times, most recently from 521d264 to 4f72c3d Compare February 2, 2023 17:18
Improves performance when there are many KXs.
Needed to debug test failures
@miki-totefu miki-totefu merged commit 30a7ec1 into companyzero:master Feb 14, 2023
@miki-totefu miki-totefu deleted the autorekx branch February 27, 2023 18:12
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.

None yet

2 participants