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

Allow disabling automatic key fetching for Olm machine #153

Merged
merged 1 commit into from
Jan 12, 2024

Conversation

hifi
Copy link
Member

@hifi hifi commented Jan 10, 2024

Many crypto operations in the Olm machine have a possible side effect of fetching keys from the server if they are missing. This may be undesired in some special cases.

To tracking which users need key fetching, CryptoStore now exposes APIs to mark and query the status.

@hifi hifi force-pushed the feature/disable-key-fetching branch from 49355bd to 20de54d Compare January 10, 2024 15:19
crypto/encryptmegolm.go Outdated Show resolved Hide resolved
crypto/sql_store.go Outdated Show resolved Hide resolved
crypto/sql_store.go Outdated Show resolved Hide resolved
crypto/sql_store.go Outdated Show resolved Hide resolved
crypto/sql_store.go Outdated Show resolved Hide resolved
@hifi hifi force-pushed the feature/disable-key-fetching branch 4 times, most recently from 41f9301 to 5ae6b22 Compare January 12, 2024 07:24
@hifi hifi marked this pull request as ready for review January 12, 2024 08:54
crypto/sql_store.go Show resolved Hide resolved
Many crypto operations in the Olm machine have a possible side effect of
fetching keys from the server if they are missing. This may be undesired
in some special cases.

To tracking which users need key fetching, CryptoStore now exposes APIs
to mark and query the status.
@hifi hifi force-pushed the feature/disable-key-fetching branch from 5ae6b22 to d51f32b Compare January 12, 2024 11:53
@hifi hifi merged commit a3883fc into master Jan 12, 2024
6 checks passed
@hifi hifi deleted the feature/disable-key-fetching branch January 12, 2024 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants