Add explicit SIGTERM listener for docker shutdown handling, fix link in readme

Author: davidje13

README.md

@@ -36,10 +36,10 @@ The output is placed in `build`.
 
 ### With Docker
 
-You can deploy using the [Docker Hub image](https://hub.docker.com/repository/docker/refacto/refacto/general):
+You can deploy using the [Docker Hub image](https://hub.docker.com/r/refacto/refacto):
 
 ```sh
-docker run -d -e INSECURE_SHARED_ACCOUNT_ENABLED=true refacto/refacto
+docker run -d -e INSECURE_SHARED_ACCOUNT_ENABLED=true -p 5000:5000 refacto/refacto
 ```
 
 (see the image details for information on how to configure and secure docker deployments).

backend/src/index.ts

@@ -73,20 +73,22 @@ function getConnectionCount(s: Server): Promise<number> {
 }
 
 let interrupted = false;
-process.on('SIGINT', async () => {
-  // SIGINT is sent twice in quick succession, so ignore the second
-  if (!interrupted) {
-    interrupted = true;
-    printInfo('');
-    try {
-      await Promise.all(shutdownTasks.map((fn) => fn()));
-      await logService.close();
-      printInfo('Shutdown complete');
-    } catch (e) {
-      printError('Failed to shutdown server', e);
-    }
+const shutdown = async () => {
+  if (interrupted) {
+    return;
   }
-});
+  interrupted = true;
+  printInfo('');
+  try {
+    await Promise.all(shutdownTasks.map((fn) => fn()));
+    await logService.close();
+    printInfo('Shutdown complete');
+  } catch (e) {
+    printError('Failed to shutdown server', e);
+  }
+};
+process.on('SIGINT', shutdown); // respond to Ctrl+C from terminal
+process.on('SIGTERM', shutdown); // e.g. sent by Docker when container is stopped
 
 (async () => {
   let success = true;