Page MenuHomePhabricator

When computing the "Subscribers" policy, use materialized membership
ClosedPublic

Authored by epriestley on Mar 13 2018, 3:27 PM.
Tags
None
Referenced Files
F18062252: D19213.id46010.diff
Mon, Aug 4, 12:38 PM
F18047416: D19213.id46009.diff
Sun, Aug 3, 11:55 AM
F18017423: D19213.id.diff
Sat, Aug 2, 6:38 PM
F17959257: D19213.diff
Fri, Aug 1, 8:01 AM
Unknown Object (File)
May 31 2025, 6:46 AM
Unknown Object (File)
May 30 2025, 6:37 PM
Unknown Object (File)
May 29 2025, 6:41 AM
Unknown Object (File)
May 10 2025, 3:08 AM
Subscribers
None

Details

Summary

Fixes T13104. The "Subscribers" policy implementation still uses older logic to query project membership and misses parent projects and milestones which a user is a member of.

Instead of doing an edge query for explicit membership, use a project query to find all projects the viewer belongs to.

Test Plan
  • Created a parent project A.
  • Created a subproject B.
  • As Bailey, created a task with "Visible To: Bailey, Subscribers".
  • Added parent project A as a task subscriber.

Then:

  • As Alice, verified I could not see the task.
  • As Alice, joined subproject B.
    • Before patch: still unable to see the task.
    • After patch: can see the task.
  • Removed parent project A as a subscriber, verified I could no longer see the task.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Mar 13 2018, 3:28 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.