The current backup key generation function using argon2 library expects &str, as the data is passed from javascript. This diffs adds an additional variant that uses u8 slices as Rust code operates on bytes. Additionally I added std feature to argon2 in this diff so it's errors implement std::error::Error. This means we can use ? operator with Box<dyn Error> in later diffs.
Details
Details
- Reviewers
marcin kamil - Commits
- rCOMMe76d33b8a73b: [native] Compute backup key for u8 slice
Summary
Test Plan
Run the provided tests.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 24 2023, 8:35 AM2023-11-24 08:35:17 (UTC-8)
Harbormaster returned this revision to the author for changes because remote builds failed.Nov 24 2023, 9:09 AM2023-11-24 09:09:18 (UTC-8)
Harbormaster completed remote builds in B24394: Diff 33593.Nov 27 2023, 12:39 AM2023-11-27 00:39:33 (UTC-8)
This revision is now accepted and ready to land.Nov 27 2023, 3:12 AM2023-11-27 03:12:59 (UTC-8)
michal added a child revision: D10054: [native] Implement native backup.Nov 27 2023, 5:38 AM2023-11-27 05:38:49 (UTC-8)
Closed by commit rCOMMe76d33b8a73b: [native] Compute backup key for u8 slice (authored by Michal Gniadek <michal.gniadek@swmansion.com>). · Explain WhyDec 1 2023, 6:19 AM2023-12-01 06:19:51 (UTC-8)
This revision was automatically updated to reflect the committed changes.