client.close() throws Error: client is closed

[brw]

Calling client.close() will always throw Error: client is closed (with UPnP at least) because the unmap calls check whether the client is closed, but they happen after this.client.close(). I imagine this is as simple as just moving the this.client.close() to the end but wasn't sure if there was more to it.

nat-port-mapper/src/index.ts

Lines 123 to 137 in 310f48c

	if (this.client != null) {
	log('Close UPnP client')
	await this.client.close()
	}
	// stop all updates
	for (const interval of this.updateIntervals.values()) {
	clearInterval(interval)
	}
	this.updateIntervals.clear()
	// Unmap all ports
	await Promise.all(
	this.openPorts.map(async opts => this.unmap(opts))
	)