Backend necessary bugfix

Author: bhasher

src/db/MongoDB.js

@@ -110,7 +110,7 @@ class MongoDB {
 
     async newLine (documentLink, previousUuid, uuid, content) {
         // Insert a line at the right place
-        //TODO is it possible in one operation ? 
+        //TODO is it possible in one operation ?
         // TODO is it possible to implement with bulk?
         try {
             let doc = await this.documentsCollection.findOne({documentLink: documentLink});
@@ -206,17 +206,18 @@ class MongoDB {
             for (let request of requests) {
                 let requestType = request.type;
                 let data = request.data;
+                let results = ""
                 switch (requestType) {
                     case 'set-line':
-                        let results = await this.setLine(documentLink, data.id, data.content);
+                        results = await this.setLine(documentLink, data.id, data.content);
                         if (!results) success = false;
                         break;
                     case 'new-line':
-                        let results = await this.newLine(documentLink, data.previous, data.id, data.content);
+                        results = await this.newLine(documentLink, data.previous, data.id, data.content);
                         if (!results) success = false;
                         break;
                     case 'delete-line':
-                        let results = await this.deleteLine(documentLink, data.id);
+                        results = await this.deleteLine(documentLink, data.id);
                         if (!results) success = false;
                         break;
                 }

src/server.js

@@ -3,7 +3,7 @@
  * @author Alexandre Dewilde
  * @date 15/11/2020
  * @version 1.0.0
- * 
+ *
  */
 const fs = require('fs');
 const path = require('path');
@@ -15,7 +15,7 @@ const port = configs.PORT;
 const DEBUG = configs.DEBUG;
 const ssl = configs.SSL;
 const sslKeyPath = configs.KEY_FILE_SSL;
-const sslCertPath = configs.CERT_FILE_SSL; 
+const sslCertPath = configs.CERT_FILE_SSL;
 
 const options = ssl ? {
     key: fs.readFileSync(sslKeyPath.startsWith('/') ? sslKeyPath : path.join(__dirname, sslKeyPath), 'utf8'),
@@ -29,7 +29,7 @@ const http = ssl ? require('https') : require('http');
 const server = http.createServer(options, app);
 if (configs.METRICS) {
     const {metricsApp} = require('./metricsApp');
-    var metricsServer = http.createServer(metricsApp);
+    var metricsServer = require('http').createServer(metricsApp);
 }
 
 // config websockets

src/socket/socket.js

@@ -66,7 +66,8 @@ module.exports = function (wss) {
 						broadcastRoomExceptSender(data, 'uuid', data.uuid);
 						const succesUpdatingDate = db.updateLastViewedDate(data.room);
 						const succesUpdate = db.applyRequests(data.room, data.data);
-						if (!succesUpdatingDate || !succesUpdate) socket.send(JSON.stringify({event: 'update', success: false}));
+						// /!\ Bad event
+						// if (!succesUpdatingDate || !succesUpdate) socket.send(JSON.stringify({event: 'update', success: false}));
 					} catch (err) {
 						if (config.DEBUG) {
 							console.error(err);
@@ -82,7 +83,7 @@ module.exports = function (wss) {
 						rooms[data.room][uuid] = socket;
 					}
 					break;
-				
+
 				case 'language':
 					try {
 						broadcastRoomExceptSender(data, 'language', data.language);