HomePhabricator
Diffusion Comm a3d27f5b40ed

[native] Fix for encrypted media

Description

[native] Fix for encrypted media

Summary:
Context can be found here: https://linear.app/comm/issue/ENG-6850/build-316-cant-load-encrypted-images

Just removing getAuthMetadata call for now and passing undefined to fetchAndDecryptMedia.

NOTE: I know that I could further "simplify" useFetchAndDecryptMedia by removing the useContext, removing the outer function, etc. but I want to keep the API the same so @bartek or whoever in the future can drop getAuthMetadata back in.

Test Plan:
Able to view encrypted images again:

001c3c.png (1×744 px, 385 KB)

Set breakpoints through fetchAndDecryptMedia and ensured that things worked as expected without auth metadata.

Reviewers: ashoat, ginsu, tomek, bartek

Reviewed By: ashoat

Subscribers: bartek

Differential Revision: https://phab.comm.dev/D11136

Details

Provenance
atulAuthored on Feb 21 2024, 11:31 AM
Reviewer
ashoat
Differential Revision
D11136: [native] Fix for encrypted media
Parents
rCOMM50d8fe4a69f2: [native] Don't dispatch an undefined token
Branches
Unknown
Tags
Unknown