kennedyshead / aioasuswrt

MIT License
24 stars 24 forks source link

Use /tmp/clientlist.json in async_get_connected_devices #81

Closed ollo69 closed 2 years ago

ollo69 commented 2 years ago

Scope of this PR is to implement the use of the file /tmp/clientlist.json to filter the list of connected device as introduced in issue #36. This should fix the problem of wireless devices that are not detected as disconnected in in new routers with new firmware. The file is used only if present and as additional filter after executing all previous check, so nothing should change if the file is not available. I tested this with AC86U, AC68U and MESH configuration for some days and everything looks fine. As you can see I have to implement a special control for wired device, to avoid problem with devices that are connected with an additional hub in cascade. This was causing the devices to appear continuously online / offline, because they appear and disappear from the file. With current implementation list of devices is really reliable at my side.

kennedyshead commented 2 years ago

Looks nice, I'll merge and test on my AC88U (should be same result as AC86U)