Merge tag '3.0.0' into develop

no message

Author: Daisuke Baba

README.md

@@ -211,6 +211,9 @@ Set `GENERIC_BLE_TRACE=true` on starting Node-RED and you can find the precise l
 
 * 3.0.0
   - Refactor entire architecture
+  - Peripheral connections are retained until it disconnects
+  - Characteristic subscriptions are retained while the ongoing flows are running (will be unsubscribed on stopping them though)
+  - The max number of concurrent BLE connections is 5 or 6 according to [this document](https://github.com/noble/noble#maximum-simultaneous-connections)
 
 * 2.0.4
   - Fix an issue where this node don't work with [email protected]

src/generic-ble.js

@@ -244,7 +244,8 @@ export default function(RED) {
       },
       unsubscribe: () => {
         return new Promise((resolve) => {
-          if (obj.notifiable) {
+          let peripheral = noble._peripherals[obj._peripheralId];
+          if (obj.notifiable && peripheral && peripheral.state === 'connected') {
             delete obj.dataListener;
             obj.object.unsubscribe(resolve);
           } else {
@@ -304,8 +305,8 @@ export default function(RED) {
                     }, []).map((c) => toCharacteristic(c));
                   });
                 });
+                peripheral.connect(); // peripheral.state => connecting
               }
-              peripheral.connect(); // peripheral.state => connecting
               connecting = true;
               break;
             }
@@ -552,6 +553,7 @@ export default function(RED) {
           }
         });
       });
+      this.emit('disconnected');
       this.on('close', (done) => {
         Object.keys(configBleDevices).forEach(k => delete configBleDevices[k]);
         this.operations.shutdown().then(done).catch(done);