Skip to content
  • epriestley's avatar
    Apply application visibility checks during normal object filtering · cf1ccc99
    epriestley authored
    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
    cf1ccc99