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

Key Backup: add passphrase support #619

Merged
merged 7 commits into from
Jan 16, 2019
Merged

Key Backup: add passphrase support #619

merged 7 commits into from
Jan 16, 2019

Conversation

manuroe
Copy link
Contributor

@manuroe manuroe commented Jan 16, 2019

Fixes element-hq/element-ios#2127.

There is also an update in restoreKeyBackup so that it fails in case of bad recovery key (or password now) instead of succeeding with 0 keys imported.

@manuroe manuroe merged commit 2513ed7 into develop Jan 16, 2019
@manuroe manuroe deleted the backup_passphrase branch January 16, 2019 14:55
Copy link

@bmarty bmarty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

little remarks

authData.publicKey = [decryption generateKey:&error];
if (password)
{
// Generate a private from the password
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"a private key"

+ (nullable NSData *)generatePrivateKeyWithPassword:(NSString*)password salt:(NSString**)salt iterations:(NSUInteger*)iterations error:(NSError * _Nullable *)error;

/**
Retrieve a private key from {password, salt, iterations)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

) -> }

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.

3 participants