Added error message when someone tries to access mailtrain while it is loading.

This prevents one to perform action on services that are still initializing (e.g. senders, where update of a send configuration causes config reload on the sender process, which may not be started yet and thus responds with error that send method is not defined)

server/app-builder.js

@@ -63,6 +63,13 @@ const interoperableErrors = require('../shared/interoperable-errors');
 const { getTrustedUrl } = require('./lib/urls');
 const { AppType } = require('../shared/app');
 
+
+let isReady = false;
+function setReady() {
+    isReady = true;
+}
+
+
 hbs.registerPartials(__dirname + '/views/partials');
 hbs.registerPartials(__dirname + '/views/subscription/partials/');
 
@@ -187,6 +194,19 @@ function createApp(appType) {
         limit: config.www.postSize
     }));
 
+
+    app.use((req, res, next) => {
+        if (isReady) {
+            next();
+        } else {
+            res.status(500);
+            res.render('error', {
+                message: 'Mailtrain is starting. Try again after a few seconds.',
+                error: {}
+            });
+        }
+    });
+
     if (appType === AppType.TRUSTED) {
         passport.setupRegularAuth(app);
     } else if (appType === AppType.SANDBOXED) {
@@ -371,3 +391,4 @@ function createApp(appType) {
 }
 
 module.exports.createApp = createApp;
+module.exports.setReady = setReady;

server/index.js

@@ -92,37 +92,36 @@ dbcheck(err => { // Check if database needs upgrading before starting the server
 */
 
     .then(() =>
-        executor.spawn(() => {
+        executor.spawn(() => 
-            testServer(() => {
+            testServer(() => 
-                verpServer(() => {
+                verpServer(() => 
-                    startHTTPServer(AppType.TRUSTED, 'trusted', trustedPort, () => {
+                    startHTTPServer(AppType.TRUSTED, 'trusted', trustedPort, () => 
-                        startHTTPServer(AppType.SANDBOXED, 'sandbox', sandboxPort, () => {
+                        startHTTPServer(AppType.SANDBOXED, 'sandbox', sandboxPort, () => 
                             startHTTPServer(AppType.PUBLIC, 'public', publicPort, () => {
                                 privilegeHelpers.dropRootPrivileges();
 
                                 tzupdate.start();
 
-                                importer.spawn(() => {
+                                importer.spawn(() => 
-                                    feedcheck.spawn(() => {
+                                    feedcheck.spawn(() => 
                                         senders.spawn(() => {
                                             triggers.start();
                                             gdprCleanup.start();
 
                                             postfixBounceServer(async () => {
-                                                (async () => {
+                                                await reportProcessor.init();
-                                                    await reportProcessor.init();
+                                                log.info('Service', 'All services started');
-                                                    log.info('Service', 'All services started');
+                                                appBuilder.setReady();
-                                                })();
                                             });
-                                        });
+                                        })
-                                    });
+                                    )
-                                });
+                                );
-                            });
+                            })
-                        });
+                        )
-                    });
+                    )
-                });
+                )
-            });
+            )
-        })
+        )
     );
 });