Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Discovery should not report expected authentication errors as errors. #23

Closed
mill1000 opened this issue Aug 30, 2023 · 0 comments · Fixed by mill1000/midea-msmart#40
Closed

Discovery should not report expected authentication errors as errors. #23

mill1000 opened this issue Aug 30, 2023 · 0 comments · Fixed by mill1000/midea-msmart#40
Assignees
@mill1000
Labels
bug Something isn't working msmart-ng Issue is with underlying msmart-ng library

Comments

@mill1000
Copy link
Owner

mill1000 commented Aug 30, 2023 •
edited
Loading

Originally noted in #22 (https://github.com/mill1000/midea-ac-py/issues/22#issuecomment-1698723281)

Part of the discovery process implemented a trial-and-error approach to determining the authentication token and key for V3 devices. This causes errors in log outputs that are expected but undesirable.
@mill1000 mill1000 added the bug Something isn't working label Aug 30, 2023
@mill1000 mill1000 self-assigned this Aug 30, 2023
@mill1000 mill1000 added the msmart-ng Issue is with underlying msmart-ng library label Aug 30, 2023
@mill1000 mill1000 mentioned this issue Aug 30, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mill1000 mill1000

Labels
bug Something isn't working msmart-ng Issue is with underlying msmart-ng library
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Supress logger messages of failed authentications during discovery mill1000/midea-msmart
1 participant
@mill1000