GeezerVison

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.28%. Comparing base (76d9f68) to head (9ad7a2d).

@@            Coverage Diff             @@
##           master    #1690      +/-   ##
==========================================
+ Coverage   93.22%   93.28%   +0.06%     
==========================================
  Files         157      157              
  Lines        9815     9844      +29     
  Branches     1003     1011       +8     
==========================================
+ Hits         9150     9183      +33     
+ Misses        472      469       -3     
+ Partials      193      192       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alexjshepler Hey, thanks for this! There are a couple of PRs out that fix the new_klap for the older IOT devices but you're right about the race when they respond on both ports. I have a more in depth PR I've been working on trying to rework the discovery process as a whole and several other things as well. I'm hopeful that soon things can start moving again on this project.

Yeah, me too. I have 3 smart switches, but when they switched to remote auth on HA two of them wouldn't auth anymore and they also removed local auth so I can't do anything but wait right now lol

Do you have an ETA for when you'll push your PR to fix the issue? I'm debating on if I want to override the integration in Home Assistant and use my fork instead

At this point, I'm waiting on the maintainers to go over everything. Once they're ready it will get merged and the. Hopefully a new version but that's out of my control.

There is already an existing, extensively tested PR that handles all of the CLI and API changes, #1580, we are working to get this implemented and a new version released soon. We will handle the discovery race detection in another PR.

Cheers for this fix; shame it got caught up in a much bigger and slower-moving PR.