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
F13267443: D7631.diff
Wed, May 29, 3:22 AM
F13252780: D7631.diff
Sat, May 25, 2:04 AM
F13230483: D7631.diff
Mon, May 20, 11:00 PM
F13229122: D7631.id17226.diff
Mon, May 20, 2:32 PM
F13177033: D7631.id17223.diff
Wed, May 8, 6:51 PM
Unknown Object (File)
Sat, May 4, 6:38 PM
Unknown Object (File)
Tue, Apr 30, 11:21 PM
Unknown Object (File)
Apr 29 2024, 7:34 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

Branch
remote_repo_uri_defaults
Lint
Lint Passed
Unit
Tests Passed

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