Merge pull request #26 from juliakm/users/jukullam/docker-updates

Updating express app and Dockerfile

Author: juliakm

app/Dockerfile

@@ -1,11 +1,14 @@
-FROM node:8
+# syntax=docker/dockerfile:1
 
-WORKDIR /usr/src/app
+FROM node:12.18.1
+ENV NODE_ENV=production
 
-COPY package*.json ./
-RUN npm ci --only=production
+WORKDIR /app
+
+COPY ["package.json", "package-lock.json*", "./"]
+
+RUN npm install --production
 
 COPY . .
 
-EXPOSE 8080
-CMD [ "npm", "start" ]
+CMD [ "node", "server.js" ]

app/server.js

@@ -1,14 +1,11 @@
-'use strict';
+const express = require('express')
+const app = express()
+const port = 3000
 
-const express = require('express');
-
-const PORT = 8080;
-const HOST = '0.0.0.0';
-
-const app = express();
 app.get('/', (req, res) => {
-  res.send('Hello world\n');
-});
+  res.send('Hello World!')
+})
 
-app.listen(PORT, HOST);
-console.log(`Running on http://${HOST}:${PORT}`);
+app.listen(port, () => {
+  console.log(`Example app listening on port ${port}`)
+})