Page MenuHomePhabricator
Differential D13991

[native][web] Handle invalid CSAT in fetchAndDecryptMedia
ClosedPublic
Actions

Authored by bartek on Thu, Nov 21, 4:00 AM.
Tags
None
Referenced Files
F3397607: D13991.id45978.diff
Sun, Dec 1, 6:28 PM
F3395926: D13991.id46057.diff
Sun, Dec 1, 9:07 AM
F3394936: D13991.id45977.diff
Sat, Nov 30, 11:52 PM
Unknown Object (File)
Fri, Nov 29, 10:21 PM
Unknown Object (File)
Fri, Nov 29, 10:12 PM
Unknown Object (File)
Fri, Nov 29, 5:35 PM
Unknown Object (File)
Fri, Nov 29, 2:49 PM
Unknown Object (File)
Wed, Nov 27, 11:34 PM
View All 16 Files
Subscribers
ashoat
tomek

Details

Reviewers
kamil
varun
ashoat
Commits
rCOMM6ae56d0edb0a: [native][web] Handle invalid CSAT in fetchAndDecryptMedia
Summary

Part of ENG-9526

Depends on D13990

Test Plan

Manual testing with mocked invalid access token

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Thu, Nov 21, 4:00 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Nov 21, 4:00 AM
bartek added a child revision: D13992: [native][web] Handle invalid CSAT in input-state-containers.Thu, Nov 21, 4:01 AM
Harbormaster failed remote builds in B32747: Diff 45922!Thu, Nov 21, 4:16 AM
bartek updated this revision to Diff 45976.Sun, Nov 24, 11:03 PM

Fix eslint

Harbormaster completed remote builds in B32789: Diff 45976.Sun, Nov 24, 11:20 PM
bartek edited the test plan for this revision. (Show Details)Sun, Nov 24, 11:24 PM
bartek published this revision for review.Mon, Nov 25, 2:06 AM
ashoat accepted this revision.Mon, Nov 25, 5:50 AM
This revision is now accepted and ready to land.Mon, Nov 25, 5:50 AM
bartek updated this revision to Diff 46057.Tue, Nov 26, 4:20 AM

Rebase

Harbormaster completed remote builds in B32842: Diff 46057.Tue, Nov 26, 9:03 AM
Closed by commit rCOMM6ae56d0edb0a: [native][web] Handle invalid CSAT in fetchAndDecryptMedia (authored by bartek). · Explain WhyFri, Nov 29, 5:03 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM6ae56d0edb0a: [native][web] Handle invalid CSAT in fetchAndDecryptMedia.

Revision Contents
Changeset List

PathSize
lib/
types/
1 line
native/
media/
20 lines
7 lines
web/
media/
28 lines

Diff 46139

View Options

lib/types/media-types.js

Loading...
View Options

native/media/encryption-utils.js

Loading...
View Options

native/media/save-media.js

Loading...
View Options

web/media/encryption-utils.js

Loading...