Homec4science

Fix a "user@domain:path" protocol handling bug

Authored by epriestley <git@epriestley.com> on Jul 28 2012, 02:21.

Description

Fix a "user@domain:path" protocol handling bug

Summary:
In D3063, we stopped converting "user@domain:path" git-style URIs, but this broke the SSH-detection code and I missed that in my test plan because my test case uses natural SSH keys so the omission of SSH flags didn't cause failures.

This code is a bit of a mess anyway. Consolidate and refactor it to be a bit simpler, and add test coverage.

Test Plan: Ran unit tests. Ran "test_connection.php" in SSH and non-SSH modes, verified SSH modes generated appropriate ssh-agent commands around the git remote commands.

Reviewers: vrana, btrahan, tberman

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T1529

Differential Revision: https://secure.phabricator.com/D3093

Details

Committed
epriestley <git@epriestley.com>Jul 28 2012, 02:21
Pushed
aubortJan 31 2017, 17:16
Parents
rPHf1eb92a12674: Fix a bug where some Maniphest batch edits were incorrectly skipped
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHd9296638cdd4: Fix a "user@domain:path" protocol handling bug (authored by epriestley <git@epriestley.com>).Jul 28 2012, 02:21