Page MenuHomePhabricator

[keyserver] Add ignorePermissions option to deleteThread
ClosedPublic

Authored by ashoat on Sep 19 2023, 1:01 PM.
Tags
None
Referenced Files
F2719166: D9229.diff
Mon, Sep 16, 12:10 PM
Unknown Object (File)
Tue, Sep 10, 12:07 PM
Unknown Object (File)
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
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
Branch
ashoat/jacek
Lint
No Lint Coverage
Unit
No Test Coverage

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