-
Notifications
You must be signed in to change notification settings - Fork 16
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
Error: spawn Unknown system error -8 after node-tunnel 4.0.0 release #39
Comments
I've been able to find the culprit, but at the moment I don't know how to fix it. The problem lies in the unzipper npm package: node-tunnel/lib/tunnel_binary.js Line 142 in 934ea2f
For some reason when it unzips the LT file inside the docker image
|
Thanks, @vrockaibrx for highlighting this issue, however, the release of a new version should not impact the older versions. We will try & validate the working package for the Linux environment. |
Yeah, I was inspecting the changes in the release and I don't see what could have caused it. It's not really a problem in the Right now my workaround is to manually unzip the LT binary into the |
Can confirm this is an issue. It's a very strange bug as well. I tried manually downloading the LT_Linux.zip file for 64 bit as defined in the config files and manually unzipped and that works. I also allowed the script to download the zip and extract it using unzipper. Strangely both extracted files have the same SHA256 sum. Stranger still they both have the same permissions. The code in node-tunnel seems fine. Can also confirm this was on node 18. |
@DanielHaroldLane they have the exact same size for me, but different sha sums (and contents). my wild guess would be a character encoding issue with unzipper npm package. |
@vrockaibrx @DanielHaroldLane Will you be able to share any sample scripts to re-produce it worked fine at my end tried it on Linux, Mac & windows as well. Also, could you please try older versions like 3.0.13 or 3.0.12 and share your findings? do give a try to the latest version as well. |
@Shivanshu-lambdatest , the LT team provided this test repo for me: |
All I did was load this repo, add an .nvmrc file in the project root and set it to lts/hydrogen. I ran I'm seeing the output: `$ mocha tests --recursive Tunnel runs successfully I've had someone else reproduce this as well. As I say, it's also happening in pipelines. The LT binary after unzip can't be executed, it seems corrupt. The zip file itself is fine if manually extracted using a zip archive tool like the linux unzip command. If you let
I think @vrockaibrx is correct. It could be a character encoding problem. I tried modifying the code to use node's inbuilt |
I've fixed this issue by using a better maintained zip npm lib. Opening a PR shortly. |
@DanielHaroldLane Thanks for the PR, version 4.0.1 has been released. This should solve the issue. |
Our pipeline stopped working properly few our ago - cca at the same time as the 4.0.0 release of the tunnel.
The error is:
Could this be related to the release?
The text was updated successfully, but these errors were encountered: