Page MenuHomePhabricator

[keyserver] Add ignorePermissions option to deleteThread
ClosedPublic

Authored by ashoat on Sep 19 2023, 1:01 PM.
Tags
None
Referenced Files
F2624488: D9229.diff
Sat, Sep 7, 1:09 AM
Unknown Object (File)
Sat, Aug 31, 1:46 PM
Unknown Object (File)
Sat, Aug 31, 8:38 AM
Unknown Object (File)
Thu, Aug 29, 4:47 AM
Unknown Object (File)
Thu, Aug 29, 1:20 AM
Unknown Object (File)
Mon, Aug 26, 7:44 PM
Unknown Object (File)
Mon, Aug 26, 6:17 PM
Unknown Object (File)
Mon, Aug 26, 5:05 PM
Subscribers

Details

Summary

This is needed in the following diff so that we can call deleteThread from a migrations context, where we use a viewer that otherwise doesn't have permissions to delete any thread (createScriptViewer(bots.commbot.userID)).

I modeled it after the equivalent permission in updateThread.

Test Plan

Before this diff, the following diff errored with an invalid_credentials error. After this diff, it no longer does.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ashoat published this revision for review.Sep 19 2023, 1:31 PM
This revision is now accepted and ready to land.Sep 19 2023, 1:35 PM