[web] Encrypt media before uploading
ClosedPublic
Actions

Authored by bartek on Apr 3 2023, 2:24 PM.

Details

Reviewers

atul
marcin

Commits

rCOMMc802440a45cd: [web] Encrypt media before uploading

Summary

This is the final diff that adds support for uploading encrypted media.

Media support in web input-state-container works differently but it's as complicated as on native. Basically: firstly, we create a pendingUpload and enhance it with actual upload result. Then (when enter/send btn is clicked) we create message based on that pending upload.

Similiar to native version, the encrypted uploads functionality is hidden behind a boolean.

Depends on D7291, D7292, D7293, D7295

Test Plan

Set the sendEncryptedMedia to false. Send a photo/video/multiple media - confirm it works.
Set the flag to true. Repeat.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.Apr 3 2023, 2:24 PM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptApr 3 2023, 2:24 PM

Harbormaster completed remote builds in B17952: Diff 24587.Apr 3 2023, 2:40 PM

bartek edited the summary of this revision. (Show Details)Apr 3 2023, 2:40 PM

bartek added a parent revision: D7295: [lib][redux] Ability to update encrypted media messages.

bartek published this revision for review.Apr 4 2023, 12:21 AM

For encryption both enabled and disabled, send a photo and confirm it works. Refresh the website to reload from keyserver.

I'm sure you tested this stack thoroughly, but generally would be good to put some more specifics on scenarios you tested in the Test Plan to give reviewers a bit more confidence. For example, did you test sending multiple encrypted photos at the same time?

Maybe a few screenshots/screen recordings to demonstrate expected behavior would also be a nice touch (though not strictly necessary)

web/input/input-state-container.react.js
305–323 ↗	(On Diff #24587)	Same feedback as previous diffs where it might improve readability to "handle the easy case first."
899–910 ↗	(On Diff #24587)	Sorry to keep adding changes that you'll need to rebase and make, but let's handle the "easy case first" here as well.