Page MenuHomePhabricator

[keyserver] Add ignorePermissions option to deleteThread
ClosedPublic

Authored by ashoat on Sep 19 2023, 1:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Feb 24, 4:26 PM
Unknown Object (File)
Sat, Feb 15, 3:54 PM
Unknown Object (File)
Thu, Feb 13, 3:52 PM
Unknown Object (File)
Thu, Feb 13, 3:52 PM
Unknown Object (File)
Thu, Feb 13, 3:51 PM
Unknown Object (File)
Jan 18 2025, 8:02 AM
Unknown Object (File)
Jan 3 2025, 1:25 PM
Unknown Object (File)
Dec 29 2024, 8:31 AM
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