Page MenuHomePhabricator

Bind patches, file content and raw diffs bind policies to their originating objects
ClosedPublic

Authored by epriestley on Dec 28 2013, 12:26 AM.

Details

Summary

Fixes T4270. When you download raw file content, diffs, and patches we currently give them default (all users) visibility.

Instead, bind them to the repository or revision in question.

(This code could use a bit of cleanup at some point.)

Test Plan

Hit the patch and content download links in Diffusion and the patch download link in Differential, got restricted files with accurate policy bindings.

Diff Detail

Branch
diffusionpolicy
Lint
Lint Passed
Unit
Tests Passed