Page MenuHomePhabricator

[lib][keyserver] Update reaction permission checks to check REACT_TO_MESSAGE
ClosedPublic

Authored by ashoat on Mar 14 2023, 8:07 PM.
Tags
None
Referenced Files
F3758083: D7074.diff
Fri, Jan 10, 8:08 AM
Unknown Object (File)
Tue, Dec 31, 9:18 AM
Unknown Object (File)
Sun, Dec 29, 7:47 PM
Unknown Object (File)
Sun, Dec 29, 7:47 PM
Unknown Object (File)
Sun, Dec 29, 7:40 PM
Unknown Object (File)
Fri, Dec 13, 10:57 AM
Unknown Object (File)
Dec 7 2024, 11:37 PM
Unknown Object (File)
Dec 6 2024, 6:11 AM
Subscribers

Details

Summary

In the last diff, I introduced a new threadPermissions constant for reactions.

In this diff, I update permission checks on the keyserver and client to check this new constant.

Depends on D7073

Test Plan

I tested this in combination with the next diff by:

  1. Running the migration by restarting the keyserver after introducing it and making sure it runs successfully
  2. Testing reacting as a member (non-admin) to a message in a COMMUNITY_ANNOUNCEMENT_ROOT community root. I don't have permission to send messages, but I can react on both native and web.

I didn't need to log out / back in on native, which indicates the the updatesTypes.UPDATE_THREAD updates were propagated correctly by the migration

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable