prepare to merge history keeper and rpc

lib/api.js

@@ -0,0 +1,60 @@
+/* jshint esversion: 6 */
+const nThen = require("nthen");
+const WebSocketServer = require('ws').Server;
+const NetfluxSrv = require('chainpad-server/NetfluxWebsocketSrv');
+
+module.exports.create = function (config) {
+    var historyKeeper;
+    var rpc;
+    const log = config.log;
+    const wsConfig = {
+        server: config.httpServer,
+    };
+
+    nThen(function (w) {
+        require('../storage/file').create(config, w(function (_store) {
+            config.store = _store;
+        }));
+    }).nThen(function (w) {
+        require("../storage/tasks").create(config, w(function (e, tasks) {
+            if (e) {
+                throw e;
+            }
+            config.tasks = tasks;
+            if (config.disableIntegratedTasks) { return; }
+
+            // XXX support stopping this interval
+            setInterval(function () {
+                tasks.runAll(function (err) {
+                    if (err) {
+                        // either TASK_CONCURRENCY or an error with tasks.list
+                        // in either case it is already logged.
+                    }
+                });
+            }, 1000 * 60 * 5); // run every five minutes
+        }));
+    }).nThen(function (w) {
+        require("./rpc").create(config, w(function (e, _rpc) {
+            if (e) {
+                w.abort();
+                throw e;
+            }
+            rpc = _rpc;
+        }));
+    }).nThen(function () {
+        var HK = require('./historyKeeper.js');
+        var hkConfig = {
+            tasks: config.tasks,
+            rpc: rpc,
+            store: config.store,
+            log: log,
+        };
+        // XXX historyKeeper exports a `setConfig` method
+        historyKeeper = HK.create(hkConfig);
+    }).nThen(function () {
+        var wsSrv = new WebSocketServer(wsConfig);
+        // XXX NetfluxSrv shares some internal functions with historyKeeper
+        // by passing them to setConfig
+        NetfluxSrv.run(wsSrv, config, historyKeeper);
+    });
+};

lib/historyKeeper.js

@@ -1,6 +1,5 @@
 /* jshint esversion: 6 */
 /* global Buffer */
-;(function () { 'use strict';
 
 const nThen = require('nthen');
 const Nacl = require('tweetnacl/nacl-fast');
@@ -63,6 +62,8 @@ const isValidValidateKeyString = function (key) {
     }
 };
 
+var CHECKPOINT_PATTERN = /^cp\|(([A-Za-z0-9+\/=]+)\|)?/;
+
 module.exports.create = function (cfg) {
     const rpc = cfg.rpc;
     const tasks = cfg.tasks;
@@ -385,8 +386,6 @@ module.exports.create = function (cfg) {
         return true;
     };
 
-    var CHECKPOINT_PATTERN = /^cp\|(([A-Za-z0-9+\/=]+)\|)?/;
-
     /*  onChannelMessage
         Determine what we should store when a message a broadcasted to a channel"
 
@@ -992,5 +991,3 @@ module.exports.create = function (cfg) {
         onDirectMessage: onDirectMessage,
     };
 };
-
-}());

server.js

@@ -4,17 +4,12 @@
 var Express = require('express');
 var Http = require('http');
 var Fs = require('fs');
-var WebSocketServer = require('ws').Server;
-var NetfluxSrv = require('chainpad-server/NetfluxWebsocketSrv');
 var Package = require('./package.json');
 var Path = require("path");
 var nThen = require("nthen");
 
 var config = require("./lib/load-config");
 
-// support multiple storage back ends
-var Storage = require('./storage/file');
-
 var app = Express();
 
 // mode can be FRESH (default), DEV, or PACKAGE
@@ -69,11 +64,9 @@ var setHeaders = (function () {
     if (Object.keys(headers).length) {
         return function (req, res) {
             const h = [
-                    /^\/pad(2)?\/inner\.html.*/,
+                    /^\/pad\/inner\.html.*/,
                     /^\/common\/onlyoffice\/.*\/index\.html.*/,
-                    /^\/sheet\/inner\.html.*/,
-                    /^\/ooslide\/inner\.html.*/,
-                    /^\/oodoc\/inner\.html.*/,
+                    /^\/(sheet|ooslide|oodoc)\/inner\.html.*/,
                 ].some((regex) => {
                     return regex.test(req.url)
                 }) ? padHeaders : headers;
@@ -117,11 +110,6 @@ app.use(function (req, res, next) {
 
 app.use(Express.static(__dirname + '/www'));
 
-Fs.exists(__dirname + "/customize", function (e) {
-    if (e) { return; }
-    console.log("Cryptpad is customizable, see customize.dist/readme.md for details");
-});
-
 // FIXME I think this is a regression caused by a recent PR
 // correct this hack without breaking the contributor's intended behaviour.
 
@@ -207,6 +195,12 @@ app.use(function (req, res, next) {
 
 var httpServer = Http.createServer(app);
 
+nThen(function (w) {
+    Fs.exists(__dirname + "/customize", w(function (e) {
+        if (e) { return; }
+        console.log("Cryptpad is customizable, see customize.dist/readme.md for details");
+    }));
+}).nThen(function (w) {
     httpServer.listen(config.httpPort,config.httpAddress,function(){
         var host = config.httpAddress;
         var hostName = !host.indexOf(':') ? '[' + host + ']' : host;
@@ -216,71 +210,21 @@ httpServer.listen(config.httpPort,config.httpAddress,function(){
 
         console.log('[%s] server available http://%s%s', new Date().toISOString(), hostName, ps);
     });
+
     if (config.httpSafePort) {
-    Http.createServer(app).listen(config.httpSafePort, config.httpAddress);
+        Http.createServer(app).listen(config.httpSafePort, config.httpAddress, w());
     }
-
+}).nThen(function () {
     var wsConfig = { server: httpServer };
 
-var rpc;
-var historyKeeper;
+    // Initialize logging then start the API server
+    require("./lib/log").create(config, function (_log) {
+        config.log = _log;
+        config.httpServer = httpServer;
 
-var log;
-
-// Initialize logging, the the store, then tasks, then rpc, then history keeper and then start the server
-var nt = nThen(function (w) {
-    // set up logger
-    var Logger = require("./lib/log");
-    //console.log("Loading logging module");
-    Logger.create(config, w(function (_log) {
-        log = config.log = _log;
-    }));
-}).nThen(function (w) {
-    if (config.externalWebsocketURL) {
-        // if you plan to use an external websocket server
-        // then you don't need to load any API services other than the logger.
-        // Just abort.
-        w.abort();
-        return;
-    }
-    Storage.create(config, w(function (_store) {
-        config.store = _store;
-    }));
-}).nThen(function (w) {
-    var Tasks = require("./storage/tasks");
-    Tasks.create(config, w(function (e, tasks) {
-        if (e) {
-            throw e;
-        }
-        config.tasks = tasks;
-        if (config.disableIntegratedTasks) { return; }
-        setInterval(function () {
-            tasks.runAll(function (err) {
-                if (err) {
-                    // either TASK_CONCURRENCY or an error with tasks.list
-                    // in either case it is already logged.
-                }
+        if (config.externalWebsocketURL) { return; }
+        require("./lib/api").create(config);
     });
-        }, 1000 * 60 * 5); // run every five minutes
-    }));
-}).nThen(function (w) {
-    require("./lib/rpc").create(config, w(function (e, _rpc) {
-        if (e) {
-            w.abort();
-            throw e;
-        }
-        rpc = _rpc;
-    }));
-}).nThen(function () {
-    var HK = require('./lib/historyKeeper.js');
-    var hkConfig = {
-        tasks: config.tasks,
-        rpc: rpc,
-        store: config.store,
-        log: log,
-    };
-    historyKeeper = HK.create(hkConfig);
-}).nThen(function () {
-    var wsSrv = new WebSocketServer(wsConfig);
-    NetfluxSrv.run(wsSrv, config, historyKeeper);
 });
+
+