-
Notifications
You must be signed in to change notification settings - Fork 3k
git+https urls don't seem to be honoured in 2.7.5 #7872
Comments
@davetayls Does this happen on |
I have a similar issue, which i can replicate on 2.7.6 as well. Note that I'm using the shrinkwrap feature, where it even though the url is directly specified in the package.json using |
Hey. Sorry for the late reply, it went mental yesterday. I'm using the shrink wrap feature as well. I'll try the build with 2.7.6 this morning and feed back. |
Seeing this on heroku as well. Forced it to use the 2.7.6 version and it still fails. |
Rolled back npm to version 2.7.3 and it works fine, so I guess it's a problem introduced in 2.7.4 |
I'm finding it quite tricky to test this without causing an issue to the testers, i think it works locally because i have the correct ssh keys for the ssh url to work. seems @kanongil and @cgreening can replicate as well which is useful 👍 thanks |
+1 Still a problem with [email protected] |
This should be fixed in [email protected] (current npm@next). Could you verify that it works for you? |
Better, but it seems be missing the credentials that should be passed in the https url
In my package json the dependencies look like:
|
This is one of the paths that I apparently didn't test enough – I can replicate that the credentials are getting stripped from the URLs and am working on a fix. |
The fix for @cgreening's issue landed in 15efe12, and after banging on it some more, I'm confident that this is something that |
Thanks very much :) |
Works on 2.8.1 Thanks! |
I'm still seeing this. What is the preferred way of downgrading npm to a specific version? |
@iMerica What version are you still seeing this on? If you want to downgrade to, say, 2.7.3 for whatever reason: $ npm install -g [email protected] |
My builds started breaking in 2.7.5 saying they don't have permission on private dependencies with the url structure:
git+https://user:[email protected]/team/repo.git
In the logs it looks like
It seems like it is not using the correct https url with the username and password.
I rolled back to using npm
2.5
and the issues went awayThe text was updated successfully, but these errors were encountered: