Page MenuHomePhabricator
Differential D13890

[native][web] Stop limiting media encryption to staff community
ClosedPublic
Actions

Authored by bartek on Thu, Nov 7, 3:41 AM.
Tags
None
Referenced Files
F3334595: D13890.id45657.diff
Thu, Nov 21, 7:11 AM
F3333397: D13890.diff
Thu, Nov 21, 4:24 AM
F3327775: D13890.id45657.diff
Wed, Nov 20, 12:20 PM
Unknown Object (File)
Mon, Nov 18, 11:18 AM
Unknown Object (File)
Sat, Nov 16, 3:07 PM
Unknown Object (File)
Fri, Nov 15, 7:13 AM
Unknown Object (File)
Fri, Nov 15, 6:38 AM
Unknown Object (File)
Fri, Nov 15, 2:16 AM
View All 25 Files
Subscribers
tomek

Details

Reviewers
ashoat
Commits
rCOMMdc9c299698c2: [native][web] Stop limiting media encryption to staff community
Summary

We had a check that enabled encrypted media only for staff community. Hardcoded it to enable encryption for all communities.

Depends on D13889

Test Plan

Used my local community for testing. Verified that after this change, multimedia messages are now of type encrypted-{photo,video}.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek created this revision.Thu, Nov 7, 3:41 AM
bartek held this revision as a draft.
Herald added a subscriber: tomek. · View Herald TranscriptThu, Nov 7, 3:41 AM
bartek added a child revision: D13891: [native][web] Flip the switch for blob-hosted media/avatar uploads.Thu, Nov 7, 3:43 AM
Harbormaster failed remote builds in B32570: Diff 45657!Thu, Nov 7, 3:44 AM
bartek mentioned this in D13892: [native][web] Clean up dead conditions for media encryption.Thu, Nov 7, 3:46 AM
Harbormaster completed remote builds in B32570: Diff 45657.Thu, Nov 7, 4:21 AM
bartek published this revision for review.Thu, Nov 7, 4:50 AM
bartek added inline comments.
native/input/input-state-container.react.js
633–636

This function is removed in D13892

ashoat accepted this revision.Thu, Nov 7, 5:32 AM
This revision is now accepted and ready to land.Thu, Nov 7, 5:32 AM
Closed by commit rCOMMdc9c299698c2: [native][web] Stop limiting media encryption to staff community (authored by bartek). · Explain WhyThu, Nov 7, 8:25 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMdc9c299698c2: [native][web] Stop limiting media encryption to staff community.
bartek mentioned this in rCOMM95e169f5657f: [native][web] Clean up dead conditions for media encryption.Fri, Nov 8, 10:23 AM

Revision Contents
Changeset List

PathSize
native/
input/
8 lines
web/
input/
8 lines

Diff 45657

View Options

native/input/input-state-container.react.js

Loading...
View Options

web/input/input-state-container.react.js

Loading...