Merge pull request #10 from pottink/master

Make node version configurable

Author: BlackIkeEagle

.env-sample

@@ -17,6 +17,14 @@ PHPVERSION=7.3
 # see https://github.com/dockerwest/nginx-symfony
 #
 NGINXVERSION=stable
+
+#
+#
+# Choose your NodeJS version. To see which versions are available see
+# https://github.com/dockerwest/nodejs
+#
+NODEVERSION=10
+
 #
 # set the symfony version, 3 and below use 3, 4 and above must use 4. Since the
 # intorduction of symfony4 there were some changes where web accessible files

bin/node

@@ -17,8 +17,8 @@ cmd = dockerrun + [
     '-v', currentpath + ":/nodeapp",
     '-e', "C_UID=" + env.get('C_UID'),
     '-e', "C_GID=" + env.get('C_GID'),
-    'dockerwest/nodejs:8',
-    'su-exec', 'node', 'node'
+    'dockerwest/nodejs:' + env.get('NODEVERSION'),
+    'node'
 ] + sys.argv[1:]
 
 os.execvp(cmd[0], cmd)

bin/npm

@@ -19,7 +19,7 @@ cmd = dockerrun + [
     '-v', currentpath + ":/nodeapp",
     '-e', "C_UID=" + env.get('C_UID'),
     '-e', "C_GID=" + env.get('C_GID'),
-    'dockerwest/nodejs:8',
+    'dockerwest/nodejs:' + env.get('NODEVERSION'),
     'npm'
 ] + sys.argv[1:]
 

bin/yarn

@@ -20,7 +20,7 @@ cmd = dockerrun + [
     '-v', currentpath + ":/nodeapp",
     '-e', "C_UID=" + env.get('C_UID'),
     '-e', "C_GID=" + env.get('C_GID'),
-    'dockerwest/nodejs:8',
+    'dockerwest/nodejs:' + env.get('NODEVERSION'),
     'yarn'
 ] + sys.argv[1:]