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

ElementR keeps re-uploading the same set of keys to backup? #26782

Closed
BillCarsonFr opened this issue Dec 18, 2023 · 6 comments · Fixed by matrix-org/matrix-rust-sdk#2957, matrix-org/matrix-js-sdk#3989 or matrix-org/matrix-js-sdk#4001
Closed

ElementR keeps re-uploading the same set of keys to backup? #26782

BillCarsonFr opened this issue Dec 18, 2023 · 6 comments · Fixed by matrix-org/matrix-rust-sdk#2957, matrix-org/matrix-js-sdk#3989 or matrix-org/matrix-js-sdk#4001
Assignees
@andybalaam
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround Z-Labs

Comments

@BillCarsonFr
Copy link
Member

https://rageshakes.element.io/api/listing/2023-12-18/103011-ZTCQB6U3/logs-0000.log.gz

2023-12-18T10:25:19.180Z I INFO matrix_sdk_crypto::backups: Successfully created a room keys backup request
 key_count=100 keys={"!LDsvXjAOHNvAHItERH:matrix.org": {"/3hJH4D8r9Va7dH9PDSgI4HX960v3ujcA3ps0LfpJU4": {"IRy0EZ9Y27MAx3f0JpkNrq2rYw0TgZ3WOPmMeXsc1Kc"}, 

2023-12-18T10:26:16.937Z I INFO matrix_sdk_crypto::backups: Successfully created a room keys backup request
    key_count=100 keys={"!LDsvXjAOHNvAHItERH:matrix.org": {"/3hJH4D8r9Va7dH9PDSgI4HX960v3ujcA3ps0LfpJU4": {"IRy0EZ9Y27MAx3f0JpkNrq2rYw0TgZ3WOPmMeXsc1Kc"}, 

2023-12-18T10:27:37.813Z I INFO matrix_sdk_crypto::backups: Successfully created a room keys backup request
    key_count=100 keys={"!LDsvXjAOHNvAHItERH:matrix.org": {"/3hJH4D8r9Va7dH9PDSgI4HX960v3ujcA3ps0LfpJU4": {"IRy0EZ9Y27MAx3f0JpkNrq2rYw0TgZ3WOPmMeXsc1Kc"}, 

2023-12-18T10:28:15.442Z I INFO matrix_sdk_crypto::backups: Successfully created a room keys backup request
    key_count=100 keys={"!LDsvXjAOHNvAHItERH:matrix.org": {"/3hJH4D8r9Va7dH9PDSgI4HX960v3ujcA3ps0LfpJU4": {"IRy0EZ9Y27MAx3f0JpkNrq2rYw0TgZ3WOPmMeXsc1Kc"}, 

2023-12-18T10:28:52.980Z I INFO matrix_sdk_crypto::backups: Successfully created a room keys backup request
    key_count=100 keys={"!LDsvXjAOHNvAHItERH:matrix.org": {"/3hJH4D8r9Va7dH9PDSgI4HX960v3ujcA3ps0LfpJU4": {"IRy0EZ9Y27MAx3f0JpkNrq2rYw0TgZ3WOPmMeXsc1Kc"},

etc..
@BillCarsonFr BillCarsonFr added the A-Element-R Issues affecting the port of Element's crypto layer to Rust label Dec 18, 2023
@ara4n ara4n changed the title ElementR keeps re-ulpoading the same set of keys to backup? ElementR keeps re-uploading the same set of keys to backup? Dec 18, 2023
@andybalaam andybalaam self-assigned this Dec 18, 2023
@florianduros florianduros added S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Dec 18, 2023
@andybalaam andybalaam mentioned this issue Dec 19, 2023
Merged
@robintown
Copy link
Member

@BillCarsonFr / @richvdh Has this fix actually made it into Element Web yet? Two weeks later, I'm seeing what appears to be the same issue judging from the console logs and high CPU usage of my client.

@andybalaam
Copy link
Member

Me too. It has not made it yet.

For my own reference: the fix was merged on 21st Dec 2023, and the latest release of matrix-rust-sdk-crypto-wasm was on 15th Dec 2023, and this is still what is referred to in matrix-js-sdk's package.json.

@andybalaam
Copy link
Member

I think I saw a comment from @richvdh that he was planning a release of matrix-rust-sdk-crypto-wasm soon. If not, I can do one.

@richvdh
Copy link
Member

richvdh commented Jan 5, 2024

Sorry, I shouldn't have let this be closed until the dependency on matrix-rust-sdk-crypto-wasm was updated.

@richvdh richvdh reopened this Jan 5, 2024
@richvdh
Copy link
Member

richvdh commented Jan 5, 2024

cc @toger5 who I think was also complaining about this recently

@richvdh richvdh mentioned this issue Jan 8, 2024
Merged
@richvdh
Copy link
Member

richvdh commented Jan 8, 2024

matrix-org/matrix-js-sdk#3989 was reverted

@richvdh richvdh reopened this Jan 8, 2024
@andybalaam andybalaam mentioned this issue Jan 10, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andybalaam andybalaam

Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience S-Major Severely degrades major functionality or product features, with no satisfactory workaround Z-Labs
Projects
None yet
Milestone
No milestone
5 participants
@andybalaam @richvdh @florianduros @BillCarsonFr @robintown