[native] Fix for encrypted media
ClosedPublic
Actions

Authored by atul on Feb 21 2024, 11:35 AM.

Details

Reviewers

ashoat
ginsu
tomek
bartek

Commits

rCOMMa3d27f5b40ed: [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:

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

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Feb 21 2024, 11:35 AM

atul published this revision for review.Feb 21 2024, 11:35 AM

atul edited the test plan for this revision. (Show Details)

ashoat accepted this revision.Feb 21 2024, 11:39 AM

This revision is now accepted and ready to land.Feb 21 2024, 11:39 AM

Harbormaster failed remote builds in B27042: Diff 37410!Feb 21 2024, 11:47 AM

We should fix the Flow issues before landing

stage all

Harbormaster completed remote builds in B27043: Diff 37411.Feb 21 2024, 12:20 PM

Closed by commit rCOMMa3d27f5b40ed: [native] Fix for encrypted media (authored by atul). · Explain WhyFeb 21 2024, 12:52 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMa3d27f5b40ed: [native] Fix for encrypted media.

Revision Contents
Changeset List

Path

Size

native/

media/

encryption-utils.js

22 lines

Diff 37411

View Options

native/media/encryption-utils.js

[native] Fix for encrypted mediaClosedPublicActions