multiple fixes for improved lifecycle and errors
* Init and _internalInit is actually awaitable * this.bleDevices is reset on init * check for actual data in write to preven length of undefined * Throttled re-inits on write-errors.
This commit is contained in:
parent
aa46579f7a
commit
126c393cee
1 changed files with 28 additions and 21 deletions
|
|
@ -85,6 +85,7 @@ class PlejdService extends EventEmitter {
|
|||
this.objectManager.removeAllListeners();
|
||||
}
|
||||
|
||||
this.bleDevices = [];
|
||||
this.connectedDevice = null;
|
||||
|
||||
this.characteristics = {
|
||||
|
|
@ -146,10 +147,7 @@ class PlejdService extends EventEmitter {
|
|||
console.log('plejd-ble: error: failed to start discovery. Make sure no other add-on is currently scanning.');
|
||||
return;
|
||||
}
|
||||
|
||||
setTimeout(async () => {
|
||||
await this._internalInit();
|
||||
}, this.connectionTimeout * 1000);
|
||||
return new Promise(resolve => setTimeout(() => resolve(this._internalInit()), this.connectionTimeout * 1000));
|
||||
}
|
||||
|
||||
async _internalInit() {
|
||||
|
|
@ -379,8 +377,20 @@ class PlejdService extends EventEmitter {
|
|||
this.characteristics.lastData.StartNotify();
|
||||
}
|
||||
|
||||
async throttledInit(delay) {
|
||||
if(this.delayedInit){
|
||||
return this.delayedInit;
|
||||
}
|
||||
this.delayedInit = new Promise((resolve) => setTimeout(async () => {
|
||||
const result = await this.init();
|
||||
this.delayedInit = null;
|
||||
resolve(result)
|
||||
}, delay))
|
||||
return this.delayedInit;
|
||||
}
|
||||
|
||||
async write(data, retry = true) {
|
||||
if (!this.plejdService || !this.characteristics.data) {
|
||||
if (!data || !this.plejdService || !this.characteristics.data) {
|
||||
return;
|
||||
}
|
||||
|
||||
|
|
@ -393,16 +403,13 @@ class PlejdService extends EventEmitter {
|
|||
setTimeout(() => this.write(data, retry), 1000);
|
||||
return;
|
||||
}
|
||||
|
||||
console.log('plejd-ble: write failed ' + err);
|
||||
setTimeout(async () => {
|
||||
await this.init();
|
||||
await this.throttledInit(this.connectionTimeout * 1000);
|
||||
|
||||
if(retry){
|
||||
logger('reconnected and retrying to write');
|
||||
await this.write(data, false);
|
||||
}
|
||||
}, this.connectionTimeout * 1000);
|
||||
}
|
||||
}
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue