Merge pull request #51 from andrewjcho84/fixmemleak

Fixmemleak

Author: andrewjcho84

app/electron/main.js

@@ -25,7 +25,8 @@ const path = require('path');
 // const fs = require('fs');
 
 console.log('NODE ENV is ', process.env.NODE_ENV);
-const isDev = process.env.NODE_ENV === 'development' || process.env.NODE_ENV === 'test' ;
+const isDev =
+  process.env.NODE_ENV === 'development' || process.env.NODE_ENV === 'test';
 const port = 8080;
 const selfHost = `http://localhost:${port}`;
 
@@ -339,8 +340,12 @@ ipcMain.on('set_cookie', event => {
   session.defaultSession.cookies
     .get({ url: serverUrl })
     .then(cookie => {
-      console.log(cookie);
-      event.reply('give_cookie', cookie);
+      // this if statement is necessary or the setInterval on main app will constantly run and will emit this event.reply, causing a memory leak
+      // checking for a cookie inside array will only emit reply when a cookie exists
+      if (cookie[0]) {
+        console.log(cookie);
+        event.reply('give_cookie', cookie);
+      }
     })
     .catch(error => {
       console.log('Error giving cookies in set_cookie:', error);