Page MenuHomePhabricator

Set reasonable defaults when displaying remote repository URIs.
ClosedPublic

Authored by frgtn on Nov 22 2013, 6:54 PM.
Tags
None
Referenced Files
F19824082: D7631.id.diff
Mar 7 2026, 7:10 PM
F19790127: D7631.id17223.diff
Feb 25 2026, 6:56 PM
F19787980: D7631.id17225.diff
Feb 25 2026, 2:39 PM
F19680663: D7631.id17223.diff
Feb 8 2026, 8:10 PM
F19671305: D7631.id17223.diff
Feb 8 2026, 12:41 AM
F19533557: D7631.id17224.diff
Jan 20 2026, 6:36 PM
F19511364: D7631.id17224.diff
Jan 12 2026, 1:12 AM
F19017566: D7631.diff
Nov 23 2025, 8:23 AM

Details

Summary

Show SSH user on git-over-ssh repositories and hide both username and password for other repos.

Test Plan

View repository details page in diffusion, Clone URI should appear with a username (taken from repo config) and any http(s) repos should be without usernames.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Update libphutil to clear that lint.

src/applications/repository/storage/PhabricatorRepository.php
584

I think $this->shouldUseSSH() is a better check. We want to show username data for explicit SSH Git, too, e.g.:

ssh://git@github.com/blah/blah
frgtn updated this revision to Unknown Object (????).Nov 22 2013, 6:58 PM
  • Remove lint.
frgtn updated this revision to Unknown Object (????).Nov 22 2013, 7:21 PM
  • Use shouldUseSSH to check for ssh uris.
epriestley closed this revision.

Closed by commit rP4910a36563a6 (authored by @frgtn, committed by @epriestley).