Typescript defs for close are missing third argument

ReconnectingWebsocket extends WebSocket but doesn’t redefine the close method to take the third “options” parameter.

Closing a reconnecting websocket like this:

rws.close(1000, '', opts);

Results in a typescript error; Expected 0-2 arguments, but got 3.

[bennycode]

I got the same issue with reconnecting-websocket v3.2.2:

Here is my code:

const Html5WebSocket = require('html5-websocket');
const ReconnectingWebsocket = require('reconnecting-websocket');

private socket: WebSocket;

// ...

public connect(clientId?: string): void { 
  this.socket = new ReconnectingWebsocket(...);
}

public disconnect(): void {
  // TS2554: Expected 0-2 arguments, but got 3.
  this.socket.close(1000, '', {keepClosed: true, fastClose: true, delay: 0});
}

Luckily there is a workaround:

public disconnect(): void {
  const socket: any = this.socket; // Cast 'WebSocket' into 'any' in order to supply a third parameter
  this.socket.close(1000, '', {keepClosed: true, fastClose: true, delay: 0});
}

Source: https://github.com/wireapp/wire-web-packages/pull/114

[bennycode]

@pladaria I checked the “close” function of v4.2.0 and it is defined to accept only 2 parameters (“code” and “reason”), so I guess there is no more support for a third (“options”) parameter?