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
F19511364: D7631.id17224.diff
Mon, Jan 12, 1:12 AM
F19017566: D7631.diff
Nov 23 2025, 8:23 AM
F18856336: D7631.id17223.diff
Nov 1 2025, 9:17 AM
F18819179: D7631.id.diff
Oct 22 2025, 4:47 AM
F18808752: D7631.diff
Oct 19 2025, 8:22 AM
F18684439: D7631.diff
Sep 26 2025, 9:32 AM
F18623345: D7631.id.diff
Sep 15 2025, 3:26 PM
F18623019: D7631.diff
Sep 15 2025, 1:57 PM

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).