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

Feat: keychain rotate passphrase #944

Merged
merged 23 commits into from
May 27, 2021
Merged
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
d04339e
changeKeychainPassword
testasdsdas May 13, 2021
f0220bb
Merge branch 'libp2p:master' into keychain-rotating-passphrase
zeim839 May 13, 2021
2591ca7
refactor: renamed changeKeychainPassword() to rotateKeychainPass()
zeim839 May 19, 2021
66eb066
refactor: renamed old/newPassword to old/newPass
testasdsdas May 19, 2021
c150749
fix: generates new options & operates on self key
testasdsdas May 19, 2021
b20e83b
doc: rotateKeychainPass
testasdsdas May 19, 2021
f8db3c9
test: rotate keychain passphrase
testasdsdas May 19, 2021
825d70c
test: rotate keychain passphrase
testasdsdas May 20, 2021
1744f8d
fix: throwDelayed error, export key with newDek
zeim839 May 21, 2021
9d591ff
lint: rotateKeychainPass
zeim839 May 21, 2021
28a2b4a
fix: rotateKeychainPass tests
zeim839 May 21, 2021
bc35ac3
fix: Uint8Array not assignable error
zeim839 May 21, 2021
87dcc5b
fix: typo
zeim839 May 22, 2021
1e06652
fix: lint
vasco-santos May 25, 2021
5e48dee
fix: lint in test
vasco-santos May 25, 2021
59e9871
fix: function returning before cycle finished
zeim839 May 26, 2021
59244c2
lint: removed parenthesis
zeim839 May 26, 2021
f855a2a
fix: await test to finish and expect to eventually be rejected
zeim839 May 26, 2021
dec59e2
fix: await test to finish and expect to eventually have property name
zeim839 May 26, 2021
0222ea7
fix: issue decrypting keys from previous tests
zeim839 May 26, 2021
ae68e1c
fix: rotateKeychainPass() test timing out
zeim839 May 26, 2021
00ed8ce
lint: fix typo
zeim839 May 26, 2021
7b66ea6
perf: removed redundant batch delete/commit
zeim839 May 26, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: await test to finish and expect to eventually be rejected
Co-authored-by: Vasco Santos <vasco.santos@ua.pt>
  • Loading branch information
zeim839 and vasco-santos committed May 26, 2021
commit f855a2ae52d0fe77f1d615b9d0d5c36c48fbaf94
3 changes: 2 additions & 1 deletion test/keychain/keychain.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -566,7 +566,8 @@ describe('keychain', () => {
: ''

// Dek with old password should not work:
expect(await kc.importKey('keyWhosePassChanged', pem, oldDek)).to.throw()
await expect(kc.importKey('keyWhosePassChanged', pem, oldDek))
.to.eventually.be.rejected()
// Dek with new password should work:
expect(await kc.importKey('keyWhosePasswordChanged', pem, newDek)).to.have.property('name', 'keyWhosePasswordChanged')
zeim839 marked this conversation as resolved.
Show resolved Hide resolved
}, 1000)
Expand Down