fix(hci): Fix peripheral connects

Author: Marco Crespi

lib/bindings/hci/Adapter.js

@@ -115,21 +115,12 @@ export class HciAdapter extends Adapter {
 			throw new Error(`Advertising and connecting simultaneously is supported with Bluetooth 4.2+`);
 		}
 
-		let timedOut = false;
 		// Create the error outside the scope to have the correct error stack
 		const timeoutError = new Error('Connecting timed out');
-		const timeout = new Promise<void>((_, reject) =>
-			setTimeout(() => {
-				timedOut = true;
-				reject(timeoutError);
-			}, 10000)
-		);
+		const timeout = new Promise<void>((_, reject) => setTimeout(() => reject(timeoutError), 10000));
 
 		const connet = async () => {
 			const handle = await this.hci.createLeConn(peripheral.address, peripheral.addressType);
-			if (timedOut) {
-				return;
-			}
 
 			this.uuidToHandle.set(peripheral.uuid, handle);
 			this.handleToUUID.set(handle, peripheral.uuid);

lib/bindings/hci/Adapter.js.map

@@ -17,6 +17,10 @@ export class HciPeripheral extends Peripheral {
 	private signaling: Signaling;
 
 	public async connect(): Promise<void> {
+		if (this._state === 'connected') {
+			return;
+		}
+
 		this._state = 'connecting';
 		await this.adapter.connect(this);
 	}
@@ -34,6 +38,10 @@ export class HciPeripheral extends Peripheral {
 	}
 
 	public async disconnect(): Promise<void> {
+		if (this._state === 'disconnected') {
+			return;
+		}
+
 		this._state = 'disconnecting';
 		await this.adapter.disconnect(this);
 	}

lib/bindings/hci/Peripheral.js

@@ -585,13 +585,6 @@ export class Hci extends EventEmitter {
 		cmd.writeUInt16LE(0x0004, 25); // min ce length
 		cmd.writeUInt16LE(0x0006, 27); // max ce length
 
-		try {
-			// Cancel any other connection requests before trying this one
-			await this.cancelLeConn();
-		} catch {
-			// NO-OP
-		}
-
 		const origScope = new Error();
 
 		return new Promise<number>(async (resolve, reject) => {