Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[PM-5693] CryptoService using memfd_secret on Linux #7
base: main
Are you sure you want to change the base?
[PM-5693] CryptoService using memfd_secret on Linux #7
Changes from 42 commits
d7c7c3e
6068e84
50dc1b4
216eb25
4b846ed
7a01168
c649bf0
b901ef7
e2129ad
f708fcc
30854f7
bed894a
709dfce
f7eda88
ce2343e
bcd712f
38343d2
f34ce02
cc27320
281619d
32d298f
c43aa08
dd37d1d
45f3d32
1f70169
22a8b17
c63f656
db3f8d4
eb81684
d279626
d5f1ede
8652d79
fdb0263
6ea6267
32088c7
f882fb2
bec4786
c2ffea6
c0d2b63
cacf4db
f4ca816
748ba75
713ec80
8f8e378
14485e8
84c0aca
f38398e
fada9c1
f9ed542
e7bfdc3
34b256e
fe6b28f
45c9ee2
ad8ed59
75b97fa
33ec078
275574f
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MissingKey2
๐ญ Will this be removed once we remove all usages of the old interfaces?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that's removed in the other PR that starts using this code in the rest of the SDK:
https://github.com/bitwarden/sdk-internal/pull/8/files#diff-3a39fb8ed55a85cd232b4ac7681c85bb81462727f6d311b1a4c33be7c5b7f4ed
I didn't want to update it here to avoid needing to touch up code which was going to need to be updated in the other PR anyway
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should these be doc comments?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've mostly added these to split up and organize all the Encryptable/Decryptable implementations.
They all look very similar and when I didn't have these comments is was quite annoying to find one in particular. Maybe what that is telling us is that we should move those impls into separate modules or separate files.
Check warning on line 49 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L43-L49
Check warning on line 61 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L55-L61
Check warning on line 73 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L67-L73
Check warning on line 85 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L79-L85
Check warning on line 100 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L93-L100
Check warning on line 113 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L106-L113
Check warning on line 125 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L119-L125
Check warning on line 137 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L131-L137
Check warning on line 149 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L143-L149
Check warning on line 161 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L155-L161
Check warning on line 182 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L174-L182
Check warning on line 201 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L193-L201
Check warning on line 220 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L214-L220
Check warning on line 237 in crates/bitwarden-crypto/src/keys/encryptable.rs
Codecov / codecov/patch
crates/bitwarden-crypto/src/keys/encryptable.rs#L231-L237