Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

ECONNREFUSED with node-sass version 3.11.1 #1787

Closed
ISO-JBA opened this issue Nov 4, 2016 · 2 comments
Closed

ECONNREFUSED with node-sass version 3.11.1 #1787

ISO-JBA opened this issue Nov 4, 2016 · 2 comments
Milestone

Comments

@ISO-JBA
Copy link

ISO-JBA commented Nov 4, 2016

  • NPM version (npm -v): 2.5.1

  • Node version (node -v): v0.12.1

  • Node Process (node -p process.versions):
    { http_parser: '2.3',
    node: '0.12.1',
    v8: '3.28.73',
    uv: '1.0.2',
    zlib: '1.2.8',
    modules: '14',
    openssl: '1.0.1m' }

  • Node Platform (node -p process.platform): win32

  • Node architecture (node -p process.arch): x64

  • node-sass version (node -p "require('node-sass').info"): error

  • npm node-sass versions (npm ls node-sass):
    └─┬ [email protected]
    └── [email protected]

Hello,

I experienced some problems with the newest version 3.11.1 of node-sass. We have a jenkins set up on an vm which builds our project. Our build fails because of the following:

Start downloading binary at https://github.com/sass/node-sass/releases/download/v3.11.1/linux-x64-46_binding.node
Cannot download "https://github.com/sass/node-sass/releases/download/v3.11.1/linux-x64-46_binding.node":
connect ECONNREFUSED 192.30.253.113:443
Hint: If github.com is not accessible in your location
try setting a proxy via HTTP_PROXY, e.g.
export HTTP_PROXY=http://example.com:1234
or configure npm proxy via
npm config set proxy http://example.com:8080

The proxy is configured on the vm and i can also download the source via wget. After adding a shrinkwrap file to our project where I define node-sass version 3.10.1 everything was fine again.
But of course I dont want to keep using the old version as a permanent solution.

I'm not sure if that has something to do with the problem but our NO_PROXY environment variable is set, so that the proxy itself would also fall under the rules.

@djusv
Copy link

djusv commented Nov 4, 2016

I was also having this issue also but I did some quick digging and I think the warning message is a bit confusing. The message states :

2016-11-03 21:53:34.347 | Hint: If github.com is not accessible in your location
2016-11-03 21:53:34.347 | try setting a proxy via HTTP_PROXY, e.g.
2016-11-03 21:53:34.347 |
2016-11-03 21:53:34.348 | export HTTP_PROXY=http://example.com:1234
But the system environment proxies are being completely ignored. Looking at merged code from #1725.

So people who before who were using system environment proxies <3.11.0 and now are upgrading to >=3.11.x will encounter this issue (timeout). I experienced this when updating gulp-sass.

xzyfer added a commit that referenced this issue Nov 5, 2016
In #1725 we started setting `proxy: ''` to defer proxy resolution
logic to request. However `request` uses `hasOwnProperty` rather
than falsey checks to determine if an option has been set. This
caused `request` to believe were configuring a proxy so it didn't
do it's own proxy resolution.

Fixes #1785
Fixes #1787
xzyfer added a commit that referenced this issue Nov 5, 2016
In #1725 we started setting `proxy: ''` to defer proxy resolution
logic to request. However `request` uses `hasOwnProperty` rather
than falsey checks to determine if an option has been set. This
caused `request` to believe were configuring a proxy so it didn't
do it's own proxy resolution.

Fixes #1785
Fixes #1787
xzyfer added a commit that referenced this issue Nov 5, 2016
In #1725 we started setting `proxy: ''` to defer proxy resolution
logic to request. However `request` uses `hasOwnProperty` rather
than falsey checks to determine if an option has been set. This
caused `request` to believe were configuring a proxy so it didn't
do it's own proxy resolution.

Fixes #1785
Fixes #1787
xzyfer added a commit that referenced this issue Nov 5, 2016
In #1725 we started setting `proxy: ''` to defer proxy resolution
logic to request. However `request` uses `hasOwnProperty` rather
than falsey checks to determine if an option has been set. This
caused `request` to believe were configuring a proxy so it didn't
do it's own proxy resolution.

Fixes #1785
Fixes #1787
@xzyfer
Copy link
Contributor

xzyfer commented Nov 5, 2016

This should be fixed with v3.11.2

@nschonni nschonni added this to the 3.11.2 milestone Nov 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants