-
Notifications
You must be signed in to change notification settings - Fork 41
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.
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
IP Discovery is not working on MacOs #158
Comments
@niemyjski can you try to use https://github.com/jlusiardi/homekit_python/tree/fix_158_unknown_categories in HASS.io or directly to do |
Yes, is there an easy way to update to this branch using pip3?
Thanks
-Blake Niemyjski
…On Sun, Aug 18, 2019 at 4:19 PM Joachim Lusiardi ***@***.***> wrote:
@niemyjski <https://github.com/niemyjski> can you try to use
https://github.com/jlusiardi/homekit_python/tree/fix_158_unknown_categories
in HASS.io or directly to do python3 -m homekit.discover --log DEBUG?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#158?email_source=notifications&email_token=AAHZFIZIU6NR6CTSATMKYNDQFG4GDA5CNFSM4IMTG4A2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4RINNA#issuecomment-522356404>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAHZFI76T7W4Y727CMGUB3LQFG4GDANCNFSM4IMTG4AQ>
.
|
Hi, |
Sweet! That worked
|
Hey @niemyjski , |
@jlusiardi in case of delays with timezones i figured i should post the original exception from the Home Assistant side of things. When trying to verify the pairing code with the TV we are seeing:
@niemyjski is running 0.97.2 here, so the homekit_python version will be 0.15.0. |
I'll try pairing it today or tomorrow. I didn't know what I needed to specify for the |
Hello,, Sorry about the delay the past week has been crazy. Here is the pairing logs where it fails.
|
To me, this looks like a completely normal M1/M2/M3 exchange, then the TV abruptly returns nothing / closes the connection instead of sending us an M4. There are no extra fields or new reserved flags suddenly being used in the M2 to indicate we should behave differently. I wondered if they rolled out a new pairing scheme (different salt? pub key?) There is no new spec, though. That likely means it failed the SRP proof? But that kind of error should (according to the spec) yield an M4 response with a |
@niemyjski and @Jc2k: i would like to close this and move the issue to #170, since this is not discovery related anymore and we seem to have the same issue with at least the Eve Energy 1EC0 plug as well. |
Fine by me! |
Continued in #170! |
I ran the following command (latest stable of everything)
I'm guessing it has to do with not supporting my LG c9 tv (wonder what it will do when I get the homekit component for my shower :D). https://www.theverge.com/2019/7/24/20708380/lg-apple-airplay-2-homekit-2019-oled-microcell-uhd-tv
The text was updated successfully, but these errors were encountered: