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.
Tags
None
Referenced Files
F18819367: D7849.id.diff
Wed, Oct 22, 5:55 AM
F18808815: D7849.diff
Sun, Oct 19, 8:51 AM
F18807738: D7849.id17758.diff
Sun, Oct 19, 3:17 AM
F18744414: D7849.id17763.diff
Fri, Oct 3, 2:53 AM
F18717505: D7849.diff
Mon, Sep 29, 2:03 PM
F18628875: D7849.diff
Sep 16 2025, 7:29 AM
F18621121: D7849.id17763.diff
Sep 15 2025, 7:08 AM
F18612097: D7849.diff
Sep 14 2025, 8:48 AM
Subscribers

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

Lint
Lint Skipped
Unit
Tests Skipped