[BUG] git+ssh does not work with version 8.16

[VeXell]

Is there an existing issue for this?

• I have searched the existing issues

This issue exists in the latest npm version

• I am using the latest npm

Current Behavior

Hello

Using the latest version of npm 8.16 and latest 14.20 version of node.js.

I have few dependencies which i install from local git server.
When i remove node_modules folder and want to install packages again with command npm i i get error:

2833 verbose stack TypeError [ERR_INVALID_URL]: Invalid URL: git+ssh://git@***:frontend/utils.git#6d45447e0c5eb6cd2e3edf05a8c5a9bb81950c79
2833 verbose stack     at new NodeError (internal/errors.js:322:7)
2833 verbose stack     at onParseError (internal/url.js:270:9)
2833 verbose stack     at new URL (internal/url.js:346:5)
2833 verbose stack     at Arborist.[registryResolved] (***/.nvm/versions/node/v14.20.0/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/arborist/reify.js:721:25)
2833 verbose stack     at Arborist.[extractOrLink] (***/.nvm/versions/node/v14.20.0/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/arborist/reify.js:641:72)
2833 verbose stack     at ***/.nvm/versions/node/v14.20.0/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/arborist/reify.js:607:35

But with npm 8.15.1 is works fine.

Expected Behavior

Packages with git+ssh path should be installed

Environment

• npm: 8.16.0
• Node.js: v14.20.0
• OS Name: MacOS 11.6.8
• System Model Name: Macbook Pro

[Delagen]

The same
npm: 8.16.0
Node.js: v18.7.0
OS Name: Windows 11

[bnpoirier]

Same here
npm: 8.17.0
Node.js: v16.16.0
OS Name: Windows 11

[Delagen]

Any news? 8.18.0 version already released

[lazybobcat]

Same here. All our pipelines fail, I had to fix all NPM versions to 8.15.

[frank-dspeed]

i even get now errors like npm exec not working with packages anymore when one includes a git dependencie is in the packages it does fail silently

[bigbossx]

throw error ERR_INVALID_URL in this line

cli/workspaces/arborist/lib/arborist/reify.js

Line 721 in fa014a5

const resolvedURL = new URL(resolved)

node URL not support parse git+ssh://

[Delagen]

@visonm I think that they need to use

const {parse}=require("node:url");

instead of global version of URL, because of Node.JS global URL is strict whatwg version, but Chromium is some patched version

[bigbossx]

@visonm I think that they need to use

const {parse}=require("node:url");

instead of global version of URL, because of Node.JS global URL is strict whatwg version, but Chromium is some patched version

You are right. you can try send a fix pr.
I'm not completely familiar with how npmcli works.