Page MenuHomePhabricator
Differential D14022

Fix an issue where paths could bleed across repos in Owners
ClosedPublic
Actions

Authored by epriestley on Aug 31 2015, 10:34 PM.
Tags
None
Referenced Files
F13179016: D14022.diff
Wed, May 8, 8:52 PM
Unknown Object (File)
Sat, May 4, 6:17 PM
Unknown Object (File)
Mon, Apr 29, 4:26 PM
Unknown Object (File)
Mon, Apr 29, 2:08 AM
Unknown Object (File)
Wed, Apr 24, 10:11 PM
Unknown Object (File)
Fri, Apr 12, 8:43 AM
Unknown Object (File)
Fri, Apr 12, 8:43 AM
Unknown Object (File)
Fri, Apr 12, 8:26 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T8320: Bring Owners out of the dark ages
Commits
Restricted Diffusion Commit
rP809c7fb4f35e: Fix an issue where paths could bleed across repos in Owners
Summary

Ref T8320. I missed this a while ago and then it came to me in a dream.

Only consider paths in the same repo when looking at ownership.

(I think this is rarely reachable in practice.)

Test Plan

Verified that files and commits still listed ownership properly.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 33907.Aug 31 2015, 10:34 PM
epriestley retitled this revision from to Fix an issue where paths coudl bleed across repos in Owners.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T8320: Bring Owners out of the dark ages.
epriestley retitled this revision from Fix an issue where paths coudl bleed across repos in Owners to Fix an issue where paths could bleed across repos in Owners.Aug 31 2015, 10:52 PM
chad accepted this revision.Sep 1 2015, 1:58 AM
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 1 2015, 1:58 AM
Closed by commit rP809c7fb4f35e: Fix an issue where paths could bleed across repos in Owners (authored by epriestley, committed by epriestley). · Explain WhySep 1 2015, 3:07 PM
This revision was automatically updated to reflect the committed changes.
• cyn0228.ch added a task: T5375: Tokenizers fail in Firefox on Android.Sep 7 2015, 10:42 PM
chad removed a task: T5375: Tokenizers fail in Firefox on Android.Sep 7 2015, 10:45 PM

Revision Contents
Changeset List

PathSize
src/
applications/
owners/
query/
5 lines

Diff 33920

View Options

src/applications/owners/query/PhabricatorOwnersPackageQuery.php

Loading...
Phabricator · Built by Phacility