All operation types and function with processing those.
Depends on D7919
Paths
| Differential D7920 Authored by kamil on May 22 2023, 10:46 AM.
Details
Summary All operation types and function with processing those. Depends on D7919 Test Plan Run tests
Diff Detail
Event Timelinekamil held this revision as a draft. Herald added subscribers: atul, ashoat. · View Herald TranscriptMay 22 2023, 10:46 AM2023-05-22 10:46:49 (UTC-7) Harbormaster completed remote builds in B19609: Diff 26802.May 22 2023, 11:04 AM2023-05-22 11:04:47 (UTC-7) This revision is now accepted and ready to land.May 24 2023, 2:06 AM2023-05-24 02:06:50 (UTC-7) Harbormaster completed remote builds in B19812: Diff 27131.May 29 2023, 4:00 AM2023-05-29 04:00:02 (UTC-7) Closed by commit rCOMM281a00b7b442: [lib] add report store ops on JS side (authored by kamil). · Explain WhyMay 29 2023, 4:18 AM2023-05-29 04:18:47 (UTC-7) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 27140 lib/ops/report-store-ops.js
lib/ops/report-store-ops.test.js
|
Before we even generate key via SymmetricKey, should we check that destination.byteLength is 32 and throw InvalidDestinationSizeException() if not?
It seems like destination is the input that we want to validate, so we should just do that directly? It's not really so much that SymmetricKey is giving us incorrect size key since it's (I'm assuming) always going to give us exactly the size we're asking for?
Feel free to re-request review if there's something I'm missing here.