fix: use same scope for setTimeout and clearTimeout calls (#1568)

Details:

- engine.io-client sets clearTimeoutFn and setTimeoutFn function
  depending on settings passed to manager

- socker.io-client is using manager.setTimeoutFn to start connection
  monitoring timer, but is using regular clearTimeout function to
  stop it when connection is established

- in some setups it is causing timer fail to stop and it will break
  connection every _timeout_ milliseconds (which is 20000 by default)

Author: mast

lib/manager.ts

@@ -129,6 +129,7 @@ export class Manager<
   // @ts-ignore
   private backoff: Backoff;
   private setTimeoutFn: typeof setTimeout;
+  private clearTimeoutFn: typeof clearTimeout;
   private _reconnection: boolean;
   private _reconnectionAttempts: number;
   private _reconnectionDelay: number;
@@ -360,8 +361,8 @@ export class Manager<
         timer.unref();
       }
 
-      this.subs.push(function subDestroy(): void {
-        clearTimeout(timer);
+      this.subs.push(() => {
+        this.clearTimeoutFn(timer);
       });
     }
 
@@ -605,8 +606,8 @@ export class Manager<
         timer.unref();
       }
 
-      this.subs.push(function subDestroy() {
-        clearTimeout(timer);
+      this.subs.push(() => {
+        this.clearTimeoutFn(timer);
       });
     }
   }