Homec4science

Apply application visibility checks during normal object filtering

Authored by epriestley <git@epriestley.com> on Jan 1 2017, 20:03.

Description

Apply application visibility checks during normal object filtering

Summary:
Fixes T9058. Normally, "Query" classes apply an application check and just don't load anything if it fails.

However, in some cases (like email recipient filtering) we run policy checks without having run a Query check first. In that case, one user (the actor) loads the object, then we filter it against other users (the recipeints).

Explicitly apply the application check during normal filtering.

Test Plan: Added a failing test case and made it pass.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T9058

Differential Revision: https://secure.phabricator.com/D17127

Details

Committed
epriestley <git@epriestley.com>Jan 2 2017, 19:00
Pushed
aubortJan 31 2017, 17:16
Parents
rPH71de5f2da255: Add more strings for Paste title changes
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHcf1ccc995e19: Apply application visibility checks during normal object filtering (authored by epriestley <git@epriestley.com>).Jan 2 2017, 19:00