Page MenuHomePhabricator

Distinguish between "Remote URI" and "Clone URI" in Repositories
ClosedPublic

Authored by epriestley on Jan 29 2014, 8:35 PM.
Tags
None
Referenced Files
F19454583: D8096.id18317.diff
Wed, Dec 31, 12:55 AM
F19356717: D8096.id18317.diff
Fri, Dec 26, 7:36 AM
F19065533: D8096.id18345.diff
Nov 30 2025, 2:29 AM
F18997432: D8096.diff
Nov 20 2025, 12:31 PM
F18844062: D8096.id18345.diff
Oct 29 2025, 2:53 AM
F18826004: D8096.id18345.diff
Oct 24 2025, 1:50 AM
F18819876: D8096.id.diff
Oct 22 2025, 10:35 AM
F18790554: D8096.id18317.diff
Oct 15 2025, 3:27 PM
Subscribers
Tokens
"The World Burns" token, awarded by zeeg."Love" token, awarded by dctrwatson.

Details

Summary

Hosted repositories have muddied this distinction somewhat. In some cases, we only want to use the real remote URI, and the call is only relevant for imported repositories.

In other cases, we want the URI we'd plug into git clone.

Move this logic into PhabricatorRepository and make the distinction more clear.

Test Plan

Viewed SVN, Git, and Mercurial hosted and remote repositories, all the URIs looked reasonable.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Jan 29 2014, 8:45 PM
  • Don't strip username from hosted SSH repositories.