Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG] GIT_SSH_COMMAND should honor GIT_SSH #129

Open
1 task done
mvorisek opened this issue Apr 14, 2023 · 0 comments
Open
1 task done

[BUG] GIT_SSH_COMMAND should honor GIT_SSH #129

mvorisek opened this issue Apr 14, 2023 · 0 comments
Labels
Needs Triage needs an initial review

Comments

@mvorisek
Copy link

mvorisek commented Apr 14, 2023 •
edited

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Currently GIT_SSH_COMMAND has hardcoded ssh path in https://github.com/npm/git/blob/v4.0.4/lib/opts.js#L5

But there is also GIT_SSH env. var honored by standard git. The GIT_SSH_COMMAND should use GIT_SSH (and quoted) instead of ssh when GIT_SSH is non-empty.

Will fix npm/cli#2891 as I can confirm npm/cli#2891 (comment) fixes the problem.

image

Expected Behavior

GIT_SSH other than ssh/empty is honored.
@mvorisek mvorisek added the Needs Triage needs an initial review label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Needs Triage needs an initial review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mvorisek