Merge pull request #292 from cmu15122/jtromero/working

third time's the charm for sockets?

Author: jacksontromero

DEPLOYMENT.md

@@ -72,6 +72,8 @@ server {
       proxy_set_header Upgrade $http_upgrade;
       proxy_set_header Connection "upgrade";
       proxy_set_header Host $host;
+      proxy_read_timeout 600s;
+      proxy_send_timeout 600s;
   }
 }
 ```

client/src/services/SocketsService.tsx

@@ -20,21 +20,24 @@ export const initiateSocket = () => {
   }
 
   socket.on('disconnect', (reason, details) => {
-    console.log('Client disconnected', reason, details);
+    console.log('Client disconnected, reconnecting', reason, details);
 
-    if (!socket.active) {
-      console.log('Inactive socket connection');
+    setTimeout(() => {
       socket.connect();
-    }
+    }, 1000);
   });
 
   socket.on('connect_error', (error) => {
-    console.log('Connection error', error.message);
-    if (!socket.active) {
-      console.log('Inactive socket connection');
+    console.log('Connection error, reconnecting', error.message);
+    setTimeout(() => {
       socket.connect();
-    }
+    }, 1000);
   });
+
+  // dumbass brute force so nginx doesn't kill us
+  setInterval(() => {
+    socket.emit('ping');
+  }, 5000);
 };
 
 export const socketSubscribeTo = (emission, callback) => {

server/controllers/sockets.js

@@ -14,6 +14,9 @@ let student_room = crypto.randomBytes(72).toString('base64');
  */
 exports.init = function (server) {
     sio = require("socket.io")(server, {
+        pingTimeout: 45000,
+        pingInterval: 2000,
+        maxHttpBufferSize: 1e7,
         cors: {
             origin: config.PROTOCOL + "://" + config.DOMAIN + ":" + config.CLIENT_PORT,
             methods: ["GET", "POST"]