Page MenuHomePhabricator

[keyserver] Skip early exit in deleteThread if ignorePermissions
ClosedPublic

Authored by ashoat on Sep 19 2023, 2:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 5, 7:10 PM
Unknown Object (File)
Wed, Jan 29, 12:07 AM
Unknown Object (File)
Wed, Jan 29, 12:07 AM
Unknown Object (File)
Wed, Jan 29, 12:06 AM
Unknown Object (File)
Wed, Jan 29, 12:03 AM
Unknown Object (File)
Dec 29 2024, 8:31 AM
Unknown Object (File)
Dec 29 2024, 8:31 AM
Unknown Object (File)
Dec 29 2024, 8:31 AM
Subscribers

Details

Summary

When I ran the migration in D9230 on production, it succeeded but didn't delete the threads. After some testing I realized there was an error in D9229, which didn't come up in my testing because I was testing in a community that commbot happened to be part of in my local environment.

The issue is that the "early exit" here will trigger if there is no permissions blob. In the case of ignorePermissions, we want the operation to go through even if there is no permissions blob for the script viewer's user (commbot).

Test Plan

I applied this patch in production, and then re-ran the migration. The bad threads were finally deleted

Diff Detail

Repository
rCOMM Comm
Branch
ashoat/jacek
Lint
No Lint Coverage
Unit
No Test Coverage