Allow specifying http and https ports via environment variables

mendhak · mendhak · commit 384a9c70b1c9 · 2020-04-09T19:19:05.000+01:00
Issue mendhak#3
diff --git a/Dockerfile b/Dockerfile
@@ -4,12 +4,11 @@ WORKDIR /app
 
 COPY . .
 
+ENV HTTP_PORT=80 HTTPS_PORT=443
+
 RUN npm install --production
 
 RUN apk --no-cache add openssl && sh generate-cert.sh && rm -rf /var/cache/apk/*
 
-EXPOSE 80 443
-
-
 ENTRYPOINT ["node", "./index.js"]
 CMD []
diff --git a/index.js b/index.js
@@ -59,8 +59,8 @@ const sslOpts = {
   cert: require('fs').readFileSync('fullchain.pem'),
 };
 
-var httpServer = http.createServer(app).listen(80);
-var httpsServer = https.createServer(sslOpts,app).listen(443);
+var httpServer = http.createServer(app).listen(process.env.HTTP_PORT || 80);
+var httpsServer = https.createServer(sslOpts,app).listen(process.env.HTTPS_PORT || 443);
 
 let calledClose = false;
 
