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

Modify encryption notice #911

Merged
merged 2 commits into from
Jan 29, 2020
Merged

Modify encryption notice #911

merged 2 commits into from
Jan 29, 2020

Conversation

bmarty
Copy link
Member

@bmarty bmarty commented Jan 28, 2020

Fixes #897

Also ensure encryption is enabled only if the algorithm is "m.megolm.v1.aes-sha2" (MXCRYPTO_ALGORITHM_MEGOLM)

@bmarty bmarty added this to the Sprint9 milestone Jan 28, 2020
@bmarty bmarty requested review from ganfra and BillCarsonFr January 28, 2020 16:27
@bmarty bmarty merged commit 305dfd4 into develop Jan 29, 2020
@bmarty bmarty deleted the feature/encryption_notice branch February 4, 2020 16:53
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.

Can we hide the algo used when turning on encryption
2 participants