Skip to content
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

proxy not forwarded to NPM #700

Open
geoHeil opened this issue Mar 5, 2018 · 6 comments
Open

proxy not forwarded to NPM #700

geoHeil opened this issue Mar 5, 2018 · 6 comments

Comments

@geoHeil
Copy link

geoHeil commented Mar 5, 2018

When using your plugin for an existing repository in a corporate setup behind a proxy it fails choose the proxy for downloading the node version.

to reproduce

git clone https://github.com/apache/ambari.git
cd ambari
mvn package

The log shows:

 Installing node version v4.5.0
[INFO] Downloading https://nodejs.org/dist/v4.5.0/node-v4.5.0-linux-x64.tar.gz to /home/vagrant/.m2/repository/com/github/eirslett/node/4.5.0/node-4.5.0-linux-x64.tar.gz
[INFO] No proxies configured
[INFO] No proxy was configured, downloading directly

and fails to download even though HTTPS_PROXY / HTTP_PROXY are set correctly.

The pom.xml ignores the proxy, but should foraward the HTTP(S)_PROXY environment variable to npm:

<configuration>
    <npmInheritsProxyConfigFromMaven>false</npmInheritsProxyConfigFromMaven>
</configuration>

@geoHeil
Copy link
Author

geoHeil commented Mar 5, 2018

Possible duplicate of #430

@Cavva79
Copy link
Contributor

Cavva79 commented Mar 20, 2018

try my fix in #683
It passes HTTPS_PROXY, HTTP_PROXY as environment variables

@Cavva79
Copy link
Contributor

Cavva79 commented May 8, 2018

@geoHeil have you solved this problem?

@geoHeil
Copy link
Author

geoHeil commented May 8, 2018 via email

@8acker
Copy link

8acker commented May 8, 2020

I had a similar issue and by providing the maven args -DproxyHost=proxy.url -DproxyPort=3128 it worked for me

@ironijunior
Copy link

I also have a similar problem.
The solution proposed by @tb0uchnafa didn`t work here, unfortunately. I hope #683 can fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants