Page MenuHomePhabricator

Fix excessively-conservative feed story policy checks
ClosedPublic

Authored by epriestley on May 9 2015, 5:53 PM.
Tags
None
Referenced Files
F14056087: D12785.diff
Sat, Nov 16, 6:18 PM
F14006539: D12785.diff
Mon, Oct 28, 12:11 PM
F13989670: D12785.diff
Mon, Oct 21, 10:17 PM
Unknown Object (File)
Oct 13 2024, 6:28 AM
Unknown Object (File)
Oct 4 2024, 1:56 PM
Unknown Object (File)
Sep 10 2024, 6:19 PM
Unknown Object (File)
Sep 4 2024, 12:42 PM
Unknown Object (File)
Aug 31 2024, 10:01 PM

Details

Summary

When checking if a user can see a feed story about an object, we currently use the object's primary policy but ignore automatic capabilities.

Instead, proxy both primary policies and automatic capabilities.

Test Plan
  • Before this patch, users could not see stories about events they were invited to but not permitted to see by the primary policy (this is currently the default for newly created events).
  • After this patch, these invited users can now see the stories.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fix excessively-conservative feed story policy checks.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a subscriber: lpriestley.
btrahan edited edge metadata.
This revision is now accepted and ready to land.May 10 2015, 5:28 PM
This revision was automatically updated to reflect the committed changes.