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.

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

Yarn not using correct SSH port for git+ssh #3146

Closed
pichouk opened this issue Apr 14, 2017 · 5 comments
Closed

Yarn not using correct SSH port for git+ssh #3146

pichouk opened this issue Apr 14, 2017 · 5 comments

Comments

@pichouk
Copy link
Contributor

pichouk commented Apr 14, 2017

Do you want to request a feature or report a bug?
A bug.

What is the current behavior?
I have some dependencies on a private Gitlab instance, mentionned like this :

"dependencies": {
    "my-package": "git+ssh://[email protected]:10202/project-name/my-package.git"
}

As you can see I use a custom port to SSH my Gitlab. It used to works perfectly but since version 0.23.2 I cannot install/update those dependencies. Yarn always use port 22 for SSH connection.

If the current behavior is a bug, please provide the steps to reproduce.
Use git+ssh dependencies on a Git server not using port 22.

What is the expected behavior?
Yarn should use the port mention on the dependency URL for SSH connection. It can be tricky since Github seems to use semicolon in private repository path...

Please mention your node.js, yarn and operating system version.
Node : v7.9.0
Yarn : 0.23.2
OS : Debian Stretch (9.0) with 4.9.0-2-amd64 Linux kernel

@timw4mail
Copy link

I'm seeing the same issue, and yarn is attempting to add a new entry in the ssh known_hosts file, resulting in a scary message.

Cleaning out known_hosts gets rid of the message, but then trying to run yarn or yarn upgrade results in a hang on the last dependency in the fetch step.

@flocsy
Copy link

flocsy commented Apr 18, 2017

I tested this and it worked in yarn v0.22.0 and is broken in v0.23.0 - v0.23.2

@pichouk
Copy link
Contributor Author

pichouk commented Apr 18, 2017

Thanks @pornel !

@pichouk
Copy link
Contributor Author

pichouk commented May 2, 2017

I just upgrade to version 0.23.3 and still get the same error. Any idea ?

@arcanis
Copy link
Member

arcanis commented May 2, 2017

The fix will be released along with the 0.24 build, which will be published somewhere this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants