A caller of the Blob API should be able to choose what needs to be uploaded, e.g. if the payload should be encrypted. This diff allows more flexibility while the old behavior isn't changed.
Depends on D9937
Details
Details
Flow.
Check if calling the new function has the same result as the old one (using some console logs, with the same initialization vectors).
Also tested if throwing exceptions while uploading produces the correct result.
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| keyserver/src/push/utils.js | ||
|---|---|---|
| 391 ↗ | (On Diff #33456) | What if encryption step throws? I have an intuition that we should put it inside try{} catch{} statement. |
| keyserver/src/push/utils.js | ||
|---|---|---|
| 391 ↗ | (On Diff #33456) | In the original code, it was outside the try-catch block, but I guess it won't hurt to put it inside. |