-
-
Notifications
You must be signed in to change notification settings - Fork 35
Open
Description
this.on('timeout', () => {
this.log.debug(
`TplinkConnection(${this.description}): timeout()`,
this.host,
this.port
);
this.queue.add(async () => {
this.close();
});
});
The promise is never triggered. So, after every timeout, a promise will be leaked in the Queue object. Note, the current close function only performs a log statement, so the actual need for a promise in the current codebase.
I discovered this on a device that I think has a dubious network connection (HS300PowerStrip) and numerous children. I did see a couple of tplink-connection objects in memory tied to that IP address, but that may have just been for the children.
Metadata
Metadata
Assignees
Labels
No labels