GitHub user necosta opened a pull request:
https://github.com/apache/zeppelin/pull/2349
[ZEPPELIN-2214] Set npm installer default npm registry
### What is this PR for?
Complements ZEPPELIN-2278, this is a fix for "connection timeout" bug when
trying to run "npm install" behind a proxy. Instead of connecting to the
official nodejs site from behind a proxy, we should use the already soft-coded
env variable ZEPPELIN_HELIUM_NPM_REGISTRY repository.
### What type of PR is it?
Bug Fix
### ToDos
* [ ] - Soft-code npm folder name as well (currently hardcoded "npm") - To
be agreed on...
### What is the Jira issue?
* [ZEPPELIN-2214 - Bug on connecting to the official nodejs site from
behind a proxy]
### How should this be tested?
Enable any Helium plugin from behind a proxy
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nokia/zeppelin zeppelin2214
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2349.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2349
----
commit 3f453c294dba8a746ef273f56e3ffb0bb8c37c2c
Author: Nelson Costa <[email protected]>
Date: 2017-05-17T14:12:58Z
[ZEPPELIN-2214] Set npm installer default npm registry
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
