-
Notifications
You must be signed in to change notification settings - Fork 344
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
Possible package integrity issue with @getgauge/cli v1.1.1 on NPM #1719
Comments
What do you see if you run this command:
Also, please check if older versions of gauge work with npm? The only significant change was e362d16#diff-da6e8c3eacb14e71fef03615908920f0 which was done to avoid Github API rate limiting. |
I see the same output i.e |
Sorry, I don't folllow.
Which output are you referring to? Older version of gauge? Also, what's the output of the command I mentioned above? |
Apologies, when i said same output, i meant the outcome was the same as my original issue i.e executing the command did nothing.
Fetching download url for Gauge version 1.1.1
4 packages are looking for funding found 0 vulnerabilities Outcome of running ./node_modules/.bin/gauge -v |
What base image is this on? Because on the official
|
this is a node 12 image. Let me try the latest node image.
Fetching download url for Gauge version 1.1.1
|
Is it possible to share your dockerfile? There seem to be some other things going on. for instance, you seem to be using Also, can you please check if you are able to install and run gauge in a vanilla node image, I mentioned the command above:
|
This is the output of the docker run against the vanila image docker run -t node /bin/sh -c "npm i @getgauge/cli; ./node_modules/.bin/gauge -v" npm ERR! A complete log of this run can be found in: |
This is new. Are you using a self signed certificate? See https://blog.npmjs.org/post/78165272245/more-help-with-selfsignedcertinchain-and-npm for options, Also see: https://stackoverflow.com/a/39453482 I suggest you try the options given above, to get gauge installed and runnning in a vanilla image. This will help setup your environment properly. You may then look at building your custom image and see if it works. |
All cert verifications by setting node_tls_auth to 0 are off and i am seeing this error now:::::::::::: (node:25) Warning: Setting the NODE_TLS_REJECT_UNAUTHORIZED environment variable to '0' makes TLS connections and HTTPS requests insecure by disabling certificate verification. Error: invalid signature: 0x4f44213c |
This happens when a there is some kind of proxy page blocking the download. So instead of a zip file it gets an HTML. Other users discuss this issue here EvanOxfeld/node-unzip#89 (comment) This really looks like an issue with a corporate proxy. |
So heres where i am a bit confused. Sounds like something in the way gauge installs has changed that doesn't allow installation through the proxy in 1.1.1.. |
Yes there's been a change in the way it downloads post 1.0.8 with the following commit e362d16#diff-e3c8a0fb0c33aadada2e90bfdb6a11e9L8-L9
This was done to avoid the rate limit on Can you try running
In your container to see what page gets downloaded? |
So that works from curl but not from npm install. Another observation 1.1.0 does not work either.. looking into the code change, it seems like the use of superagent seems to be leaving out the proxies that are being sent. Similar sort of issue reported against superagent: Also to be raised is the fact that in 1.0.8 - i was seeing useful error messages when i was hitting certification or proxy issues |
|
Signed-off-by: sriv <[email protected]>
Running a gauge v1.1.1 install on node 12 does not install properly
Things to bear in mind:
Post installation when i run - gauge -v --> I dont get anything, not even an error
This means that the command gauge is recognized but does nothing
Hurdles faced in getting to this point:
I felt the need to raise this as a bug as this issue has cropped up a few times and it has been closed without resolution.
refer: #1666
#1683
The text was updated successfully, but these errors were encountered: