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.

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

Add error for wrong HomeKit source #805

Closed
metril opened this issue Dec 7, 2023 · 7 comments
Closed

Add error for wrong HomeKit source #805

metril opened this issue Dec 7, 2023 · 7 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@metril
Copy link

metril commented Dec 7, 2023

I am running v1.8.4 of the go2rtc addon in Home Assistant 2023.12.

I have added the Aqara G4 Doorbell to Home Assistant via Homekit.

My go2rtc config looks like the following:

streams:
  aqara_g4_doorbell: homekit://some_address_here_copied_from_Add_tab_in_go2rtc
  
rtsp:
  listen: ":8554"
  username: "someusername"
  password: "somepassword"

webrtc:
  listen: ":9555"

homekit:
  aqara_g4_doorbell:
    pin: 12345678
    name: Aqara G4 Doorbell
    device_id: aqara_g4_doorbell
    device_private: aqara_g4_doorbell

I can see the Aqara G4 Doorbell in the Home app on IOS. I go to add it and then I get an error, unable to add accessory. In the go2rtc logs, I see:

ERR github.com/AlexxIT/go2rtc/internal/homekit/homekit.go:179 > error="ed25519: invalid params"

I expect to be able to proxy and successfully add the Aqara G4 Doorbell to the IOS Home app.

@metril metril changed the title Unable to proxy Aqara G4 Doorbell Unable to proxy Aqara G4 Doorbell and add to the Home app Dec 7, 2023
@AlexxIT AlexxIT added the question Further information is requested label Dec 7, 2023
@AlexxIT
Copy link
Owner

AlexxIT commented Dec 7, 2023

Try to use another pin. Something more random.

@metril
Copy link
Author

metril commented Dec 7, 2023

Same error. I've tried default pin and several random ones.

@AlexxIT
Copy link
Owner

AlexxIT commented Dec 7, 2023

You should have pairings list in the go2rtc config. As I can see, you don't have it in your config.
Your error from Verify step. This means that Pair strep was successful. After pair step you should see pairing keys in config.

@metril
Copy link
Author

metril commented Dec 7, 2023

When I try to pair, I do see the pair key loaded into my go2rtc config. However, the IOS app complains that there was an error and then I don't see any new device.

Error in IOS app: "Unable to add accessory. An unknown error occurred."

@metril
Copy link
Author

metril commented Dec 10, 2023

I figured it out. There is some documentation that is missing. I also came across the Home Assistant forum post where someone had a similar issue and got it resolved. That post was also missing crucial information.

When you have the Aqara G4 Doorbell added to Home Assistant via the homekit integration, you need to then use the "homekit" URL listed under "Home Assistant" in the "Add" tab. If you use the homekit URL listed until "Homekit", then it will not work and you will get the error in the IOS Home App that I was getting.

@AlexxIT
Copy link
Owner

AlexxIT commented Dec 11, 2023

Yes, you're right. The HomeKit section is used to peer new devices. But your device is already connected to Hass. I need to rewrite error.

@AlexxIT AlexxIT self-assigned this Dec 11, 2023
@AlexxIT AlexxIT added enhancement New feature or request and removed question Further information is requested labels Dec 11, 2023
AlexxIT added a commit that referenced this issue Dec 30, 2023
@AlexxIT AlexxIT added this to the v1.8.5 milestone Dec 30, 2023
@AlexxIT AlexxIT changed the title Unable to proxy Aqara G4 Doorbell and add to the Home app Add error for wrong HomeKit source Jan 1, 2024
@AlexxIT
Copy link
Owner

AlexxIT commented Jan 1, 2024

@AlexxIT AlexxIT closed this as completed Jan 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants