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.

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

Leader client #2402

Closed
bedhub opened this issue Nov 3, 2020 · 3 comments
Closed

Leader client #2402

bedhub opened this issue Nov 3, 2020 · 3 comments
Assignees
@mpfau
Labels
improvement nice-to-haves that are not impeding usage of any features state:tested We tested it and are about to release it
Milestone
3.78.0

Comments

@bedhub
Copy link
Contributor

bedhub commented Nov 3, 2020

Some task are not required to execute by all connected client. There should be a leader status for the client which is responsible for processing inbox rules, updating entropy on TutanotaProperties and updating ownerEncSession key.
@bedhub bedhub added the bug broken functionality, usability problems, unexpected errors label Nov 3, 2020
@bedhub bedhub added this to the 3.78.0 milestone Nov 3, 2020
@bedhub
Copy link
Contributor Author

bedhub commented Nov 12, 2020

See commits: e44567d 31f5307

@bedhub bedhub closed this as completed Nov 12, 2020
charlag added a commit that referenced this issue Nov 13, 2020
@charlag
Fix CryptoFacadeTest, #2402
782b7c7 
cherry-pick'ed from entity-update-lookahead branch
@charlag charlag mentioned this issue Nov 13, 2020
Merged
bedhub pushed a commit that referenced this issue Nov 13, 2020
@charlag @bedhub
Fix CryptoFacadeTest, #2402
49c0d28 
cherry-pick'ed from entity-update-lookahead branch
@charlag charlag added improvement nice-to-haves that are not impeding usage of any features and removed bug broken functionality, usability problems, unexpected errors labels Nov 19, 2020
@bedhub bedhub added improvement nice-to-haves that are not impeding usage of any features and removed improvement nice-to-haves that are not impeding usage of any features labels Nov 19, 2020
@bedhub
Copy link
Contributor Author

bedhub commented Nov 20, 2020 •
edited

  • Only the leader client executes inbox rules for new emails received when logged in
  • Non leader client may still process inbox rules when explicitly selecting the inbox folder
  • Only the leader client updates the owner enc session key
  • Only the leader client updates entropy
  • Switching of leading client (new client versions are preferred)
  • Check that old clients can receive push notification without errors

@mpfau
Copy link
Contributor

mpfau commented Nov 20, 2020

  • non leaders still download incoming mails immediately
  • desktop client push does not work for me

@bedhub bedhub added the state:tested We tested it and are about to release it label Nov 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mpfau mpfau

Labels
improvement nice-to-haves that are not impeding usage of any features state:tested We tested it and are about to release it
Projects
None yet
Milestone
3.78.0
Development

No branches or pull requests
3 participants
@mpfau @charlag @bedhub