Added closing of event listener

Signed-off-by: French Ben <me+git@frenchben.com>

src/app.js

@@ -66,8 +66,14 @@ setupUtil.setup().then(() => {
   throw err;
 });
 
+
 ipcRenderer.on('application:quitting', () => {
+  docker.detachEvent();
   if (localStorage.getItem('settings.closeVMOnQuit') === 'true') {
     machine.stop();
   }
 });
+
+window.onbeforeunload = function () {
+  docker.detachEvent();
+};

src/browser.js

@@ -47,6 +47,7 @@ app.on('ready', function () {
     return false;
   });
 
+
   if (os.platform() === 'win32') {
     mainWindow.on('close', function () {
       mainWindow.webContents.send('application:quitting');

src/utils/ContainerUtil.js

@@ -33,10 +33,12 @@ var ContainerUtil = {
       var [dockerPort, portType] = key.split('/');
       var localUrl = null;
       var port = null;
-
       if (value && value.length) {
         port = value[0].HostPort;
       }
+      if (ip === 'docker.local') {
+        ip = value[0].HostIp;
+      }
       localUrl = (port) ? ip + ':' + port : ip + ':' + '<not set>';
 
       res[dockerPort] = {

src/utils/DockerUtil.js

@@ -19,7 +19,8 @@ export default {
   host: null,
   client: null,
   placeholders: {},
-  streams: {},
+  stream: null,
+  eventStream: null,
   activeContainerName: null,
 
   setup (ip, name) {
@@ -377,7 +378,7 @@ export default {
   },
 
   active (name) {
-    this.detach();
+    this.detachLog();
     this.activeContainerName = name;
 
     if (name) {
@@ -431,9 +432,7 @@ export default {
         return;
       }
 
-      if (this.stream) {
-        this.detach();
-      }
+      this.detachLog()
       this.stream = logStream;
 
       let timeout = null;
@@ -452,14 +451,22 @@ export default {
     });
   },
 
-  detach () {
+  detachLog() {
     if (this.stream) {
       this.stream.destroy();
       this.stream = null;
     }
   },
+  detachEvent() {
+    if (this.eventStream) {
+      this.eventStream.destroy();
+      this.eventStream = null;
+    }
+  },
+
 
   listen () {
+    this.detachEvent()
     this.client.getEvents((error, stream) => {
       if (error || !stream) {
         // TODO: Add app-wide error handler
@@ -474,13 +481,13 @@ export default {
 
         if (data.status === 'destroy') {
           containerServerActions.destroyed({id: data.id});
-          this.detach(data.id);
+          this.detachLog()
         } else if (data.status === 'kill') {
           containerServerActions.kill({id: data.id});
-          this.detach(data.id);
+          this.detachLog()
         } else if (data.status === 'stop') {
           containerServerActions.stopped({id: data.id});
-          this.detach(data.id);
+          this.detachLog()
         } else if (data.status === 'create') {
           this.logs();
           this.fetchContainer(data.id);
@@ -491,6 +498,7 @@ export default {
           this.fetchContainer(data.id);
         }
       });
+      this.eventStream = stream;
     });
   },