Page MenuHomePhabricator
Differential D13890

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

Authored by bartek on Nov 7 2024, 3:41 AM.
Tags
None
Referenced Files
F3522885: D13890.id45695.diff
Mon, Dec 23, 7:50 AM
F3522884: D13890.id45657.diff
Mon, Dec 23, 7:50 AM
F3522874: D13890.id.diff
Mon, Dec 23, 7:50 AM
F3522871: D13890.diff
Mon, Dec 23, 7:50 AM
Unknown Object (File)
Fri, Nov 29, 2:26 PM
Unknown Object (File)
Fri, Nov 29, 6:14 AM
Unknown Object (File)
Thu, Nov 28, 11:12 PM
Unknown Object (File)
Nov 24 2024, 3:21 AM
View All 33 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.Nov 7 2024, 3:41 AM
bartek held this revision as a draft.
Herald added a subscriber: tomek. · View Herald TranscriptNov 7 2024, 3:41 AM
bartek added a child revision: D13891: [native][web] Flip the switch for blob-hosted media/avatar uploads.Nov 7 2024, 3:43 AM
Harbormaster failed remote builds in B32570: Diff 45657!Nov 7 2024, 3:44 AM
bartek mentioned this in D13892: [native][web] Clean up dead conditions for media encryption.Nov 7 2024, 3:46 AM
Harbormaster completed remote builds in B32570: Diff 45657.Nov 7 2024, 4:21 AM
bartek published this revision for review.Nov 7 2024, 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.Nov 7 2024, 5:32 AM
This revision is now accepted and ready to land.Nov 7 2024, 5:32 AM
Closed by commit rCOMMdc9c299698c2: [native][web] Stop limiting media encryption to staff community (authored by bartek). · Explain WhyNov 7 2024, 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.Nov 8 2024, 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...