cyberjunky / home-assistant-arpscan_tracker

This component tracks devices using the arp-scan liinux command, it's very fast, and reasonably accurate.
MIT License
28 stars 4 forks source link

Fix issue with new `include_host` #6

Closed DeadEnded closed 3 years ago

DeadEnded commented 3 years ago

Removed white space on empty line def get_device_name (line 76).

Added a check to see if include_host is empty before excluding devices not in the list. This is to fix a bug (#5) where if there is no include_host flag, all devices are excluded. I have tested this in my installation, and it appears to fix the issue - there may be a better way to do this check than nesting another if statement.

cyberjunky commented 3 years ago

@DeadEnded Thanks!