From 03425d7a3f27a466140f3c25968fda9a026a4feb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lucas=20Mind=C3=AAllo=20de=20Andrade?= Date: Sat, 17 Aug 2024 13:23:16 -0300 Subject: [PATCH] chore(device): improved authentication logs (#276) ## Summary by CodeRabbit - **Enhancements** - Improved logging functionality for the authentication and device enabling processes, providing better visibility and traceability of actions taken on devices. --- midealocal/device.py | 23 ++++++++++++++++++++--- 1 file changed, 20 insertions(+), 3 deletions(-) diff --git a/midealocal/device.py b/midealocal/device.py index 26350d44..4a2bc460 100644 --- a/midealocal/device.py +++ b/midealocal/device.py @@ -8,6 +8,8 @@ from enum import IntEnum, StrEnum from typing import Any +from typing_extensions import deprecated + from .exceptions import CannotConnect, SocketException from .message import ( MessageApplianceResponse, @@ -245,6 +247,12 @@ def authenticate(self) -> None: raise SocketException self._socket.send(request) response = self._socket.recv(512) + _LOGGER.debug( + "[%s] Received auth response with %d bytes: %s", + self._device_id, + len(response), + response.hex(), + ) if len(response) < MIN_AUTH_RESPONSE: self.enable_device(False) raise AuthException @@ -460,13 +468,22 @@ def update_all(self, status: dict[str, Any]) -> None: for update in self._updates: update(status) - def enable_device(self, available: bool = True) -> None: - """Enable device.""" - _LOGGER.debug("[%s] Enabling device", self._device_id) + def set_available(self, available: bool = True) -> None: + """Set available value.""" + _LOGGER.debug( + "[%s] %s device", + self._device_id, + "Enabling" if available else "Disabling", + ) self._available = available status = {"available": available} self.update_all(status) + @deprecated("enable_device is replaced by set_available") + def enable_device(self, available: bool = True) -> None: + """Enable device.""" + self.set_available(available) + def open(self) -> None: """Open thread.""" if not self._is_run: