Merge pull request #476 from Coding/terminal-reconnect-fix

fix terminal reconnect bug

Author: KevenYoung

app/backendAPI/websocketClients.js

@@ -117,7 +117,6 @@ class TtySocketClient {
     } else {
       this.socket = io.connect(config.baseURL, { resource: 'coding-ide-tty1' })
     }
-
     this.backoff = getBackoff({
       delayMin: 1500,
       delayMax: 10000,
@@ -160,7 +159,6 @@ class TtySocketClient {
   }
 
   reconnect () {
-    log(`[TTY Socket] Try reconnect ttySocket ${this.backoff.attempts}`)
     if (this.backoff.attempts <= this.maxAttempts && !this.socket.connected) {
       const timer = setTimeout(() => {
         this.connect()

app/components/Terminal/terminal-client.js

@@ -92,7 +92,7 @@ class TerminalClient extends TtySocketClient {
     })
 
     this.socket.on('disconnect', (type) => {
-      console.log('terminal disconnect...')
+      console.log('[TTY Socket] Terminal disconnect...')
       if (config.ttySocketConnected) runInAction(() => config.ttySocketConnected = false)
       if (type !== 'manual') {
         this.reconnect()
@@ -166,7 +166,7 @@ class TerminalClient extends TtySocketClient {
       // this.socket.disconnect()
       this.close()
       if (this.unbindSocketEvent) this.unbindSocketEvent()
-      this.socket = null
+      // this.socket = null ??
       SideBar.hideSidePanelView('SIDEBAR.BOTTOM.terminal')
     }
   }