Page MenuHomePhabricator
Differential D10855

Diffusion - add verifyGitOrigin check to git fetch operation
ClosedPublic
Actions

Authored by btrahan on Nov 15 2014, 1:21 AM.
Tags
None
Referenced Files
F18756808: D10855.id.diff
Sun, Oct 5, 1:39 PM
F18746226: D10855.diff
Fri, Oct 3, 11:52 AM
F18716008: D10855.diff
Mon, Sep 29, 11:18 AM
F18672686: D10855.diff
Thu, Sep 25, 8:04 AM
F18626255: D10855.id26062.diff
Tue, Sep 16, 12:52 AM
F18597422: D10855.diff
Sat, Sep 13, 2:51 AM
F18585814: D10855.diff
Thu, Sep 11, 4:29 PM
F18504405: D10855.id.diff
Sep 4 2025, 11:47 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T4946: Git checks for missing origin in 1.7.1 happen too late in the update process
T5938: Unable to setup a clone of private git repo hosted on GitHub
Commits
Restricted Diffusion Commit
rPd2ea0bc5f0df: Diffusion - add verifyGitOrigin check to git fetch operation
Summary

Fixes T4946. Theoretically.

Test Plan

iiam

also unit tests.

also

cd /var/repo/X
git remote remove origin # simulates origin-missing clone under 1.7.1
cd /path/to/phabricator
./bin/repository pull X

and observed no errors

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 26062.Nov 15 2014, 1:21 AM
btrahan added tasks: T5938: Unable to setup a clone of private git repo hosted on GitHub, T4946: Git checks for missing origin in 1.7.1 happen too late in the update process.
btrahan retitled this revision from to Diffusion - add verifyGitOrigin check to git fetch operation.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptNov 15 2014, 1:21 AM
btrahan edited the test plan for this revision. (Show Details)Nov 15 2014, 1:21 AM
Harbormaster completed remote builds in B3056: Diff 26062.Nov 15 2014, 1:21 AM
epriestley accepted this revision.Nov 15 2014, 1:23 AM
epriestley edited edge metadata.

You can probably test by running:

cd /var/repo/X
git remote remove origin # simulates origin-missing clone under 1.7.1
cd /path/to/phabricator
./bin/repository pull X
This revision is now accepted and ready to land.Nov 15 2014, 1:23 AM
btrahan edited the test plan for this revision. (Show Details)Nov 15 2014, 2:02 AM
btrahan edited edge metadata.
Closed by commit rPd2ea0bc5f0df: Diffusion - add verifyGitOrigin check to git fetch operation (authored by btrahan, committed by btrahan). · Explain WhyNov 15 2014, 2:02 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 26063

View Options

src/applications/repository/engine/PhabricatorRepositoryDiscoveryEngine.php

Loading...
View Options

src/applications/repository/engine/PhabricatorRepositoryEngine.php

Loading...
View Options

src/applications/repository/engine/PhabricatorRepositoryPullEngine.php

Loading...
Phabricator · Built by Phacility