BREAKING CHANGE: autoDiscoverAll is not defaulting to false for faster device connexion

farfromrefug · farfromrefug · commit be5d9118e5aa · 2022-01-20T13:53:54.000+01:00
diff --git a/src/bluetooth.common.ts b/src/bluetooth.common.ts
@@ -351,7 +351,7 @@ export interface ConnectOptions {
     onDisconnected?: (data: { UUID; name: string }) => void;
 
     /**
-     * Discover all services on connection
+     * Discover all services on connection. Default is false for faster connection
      */
     autoDiscoverAll?: boolean;
 
diff --git a/src/bluetooth.ios.ts b/src/bluetooth.ios.ts
@@ -995,7 +995,7 @@ export class Bluetooth extends BluetoothCommon {
                     this.centralManager.connectPeripheralOptions(peripheral, null);
                 });
                 let services, mtu = FIXED_IOS_MTU;
-                if (args.autoDiscoverAll !== false) {
+                if (args.autoDiscoverAll === true) {
                     services = (await this.discoverAll({ peripheralUUID: connectingUUID }))?.services;
                 }
                 if (!!args.autoMaxMTU) {

Original file line number	Diff line number	Diff line change
`@@ -995,7 +995,7 @@ export class Bluetooth extends BluetoothCommon {`
`995`	`995`	`this.centralManager.connectPeripheralOptions(peripheral, null);`
`996`	`996`	`});`
`997`	`997`	`let services, mtu = FIXED_IOS_MTU;`
`998`		`- if (args.autoDiscoverAll !== false) {`
	`998`	`+ if (args.autoDiscoverAll === true) {`
`999`	`999`	`services = (await this.discoverAll({ peripheralUUID: connectingUUID }))?.services;`
`1000`	`1000`	`}`
`1001`	`1001`	`if (!!args.autoMaxMTU) {`