chore: fix clean release.

The Dockerfile has an "optional" COPY statement that copies over generated development SSL certificates. However COPY fails when given no inputs. So on a clean repo it currently doesn't work. I'm removing the COPY statement from the Dockerfile and instead using (another) volume mount for that. Unfortunately that does require the certificate files to live in a subdirectory.

Author: pieterpaulkramer

.gitignore

@@ -1,5 +1,3 @@
 node_modules/
+cert/
 package-lock.json
-cert.pem
-key.pem
-csr.pem

Dockerfile

@@ -9,8 +9,5 @@ RUN yarn install --prod --immutable
 COPY *.js ./
 COPY public public/
 
-# SSL certificates. For testing only: doesn't match any files on a clean repo.
-COPY *.pem ./
-
 EXPOSE 3000 9001 10000-11000
 CMD ["node", "main.js"]

README.md

@@ -22,7 +22,7 @@ For local development you need to generate a self-signed certificate as voice me
 
 With those files, you can run the server using:
 
-    export SSL_CERT_PATH="./cert.pem" SSL_KEY_PATH="./key.pem"
+    export SSL_CERT_PATH="./cert/cert.pem" SSL_KEY_PATH="./cert/key.pem"
 
 Or to wherever the key files are located on your production server.
 
@@ -35,7 +35,7 @@ For running in production, make sure to set the `ANNOUNCED_IP` environmental var
 Proximity Chat can run in a Docker container. Use the following steps to build and run using Docker:
 
     docker build -t proximity-chat .
-    docker run -d -p 3000:3000 -p 9001:9001 -v "/$(pwd)\public":/usr/src/app/public:ro -e SSL_CERT_PATH="./cert.pem" -e SSL_KEY_PATH="./key.pem" proximity-chat
+    docker run -d -p 3000:3000 -p 9001:9001 -v "/$(pwd)\public":/usr/src/app/public:ro -v "/$(pwd)\cert":/usr/src/app/cert:ro -e SSL_CERT_PATH="./cert/cert.pem" -e SSL_KEY_PATH="./cert/key.pem" proximity-chat
 
 This also binds your local `public` directory to the container, so that you can test frontend changes more quickly.
 

package.json

@@ -7,7 +7,7 @@
     "start": "node --trace-warnings main.js",
     "lint": "eslint .",
     "lint-fix": "eslint . --fix",
-    "gen-ssl-certs": "openssl req -new -newkey rsa:2048 -nodes -keyout key.pem -x509 -out cert.pem -days 365 -subj /C=NL/ST=NB/L=Eindhoven/O=Disc/CN=www.demos.nl"
+    "gen-ssl-certs": "openssl req -new -newkey rsa:2048 -nodes -keyout ./cert/key.pem -x509 -out ./cert/cert.pem -days 365 -subj /C=NL/ST=NB/L=Eindhoven/O=Disc/CN=www.demos.nl"
   },
   "author": "Meshiest",
   "license": "CC0 1.0 Universal",