fwestenberg / reolink_dev

Home Assistant Reolink addon
MIT License
550 stars 101 forks source link

Error when adding Reolink RLC-820A #463

Closed gdeuss closed 2 years ago

gdeuss commented 2 years ago

Getting the following integration error when attempting to add my first Reolink Camera to Home Assitant

Please let me know if I missed any info required to review this, I am not sure what else you may need to review. Running Home Assistant Core 2022.3.3 with Supervisor 2022.3.2 on HA OS 7.4 with Reolink V0.50

image

Obviously, the IP and the username password must be ok

image

image

klu03rl commented 2 years ago

did you have this issue after upgrading the firmware? can you advise the firmware version you are using - I am having issue after upgrading today

gdeuss commented 2 years ago

No, never was able to connect. The firmware it shows on my setup is currently v3.0.0.494. It tells me that is the latest but when checking the Reolink website that latest seems to be v3.1.0.764. If you want me to use a specific version, let me know, happy to try. Trouble is, that is only one of 4 different cameras I use ☹

gdeuss commented 2 years ago

Just used the same camera type with integration v3.1.0.764. with the following result see below. Different but this time it can not even connect to the camera. No entry in the log for that time. It shows port 443 once the msg appears but I actually use port 80 with HTTPS disabled. With firmware v3.0.0.494 using port 80 with HTTP only or port 443 with HTTPS disabled produces the same result

image

cpainchaud commented 2 years ago

Integration won't work with 2022.3 HA until I (or someone else) submits a patch. I need to find some time to work on it this week.

gdeuss commented 2 years ago

Thanks, Christophe, I assume that would apply to all Reolink camera types, correct?

cpainchaud commented 2 years ago

All fo them, HA has changed their APIs so it requires a fix somewhere to cope with that general change

cpainchaud commented 2 years ago

please try 0.51 , now please note that if you keep mixing internal non-HTTP with external HTTPS address, it's not supported by HomeAssistant themselves

gdeuss commented 2 years ago

@cpainchaud Thanks for the update, I could add three out of six cameras after the update and at least the Lovelace video integration works great now. The other three cameras do not work and show the same issue as before with "Failed to connect with camera" or "Unexpected error". Will document that in more detail later today. I noticed that I had two RLC-820A cameras, one was on firmware .494 (older firmware) and the other was on .764 (latest firmware). The camera with the .494 fimrware worked right away and the one with the .764 software did not connect. I loaded the .494 on the second camera and I was able to integrate the camera immediately. I will post the error logs for the remaining cameras later today. Unfortunately, downgrading the firmware on some of the cameras won't work in all cases due to other, camera-related issues for some of the models. The models currently not working for me are RLC-823A ("Unexpected error") and RLC-1212A ("Failed to connect with the camera"). Models RLC-820A and E1 Zoom (not supported according to the HA integration documentation) are working fine

gdeuss commented 2 years ago

@cpainchaud @ Did some more testing with another camera model RLC-510WA and it looks like the remaining issues with the video integration are related to the .764 updates on the cameras. When I rolled back the camera firmware on hat camera to .494 the camera vidoe feed started working instantly. The other two remaining camera types I have are RLC-823A and RLC-1212A. Neither has previous firmware available. They are new camera models and I can not roll them back. Is it possible that Reolink made some changes to their API with their .764 and later updates that the current Reolink .51 has not yet accounted for? There is s separate but possibly related issue with the sensor integration of all cameras and I have not done any search for that issue yet and it has possibly been reported already. The sensor values got to "unavailable" after a while if you log out of HA and they will stay unavailable until to log back into HA. They will start updating again shortly after you log in and stay available as long as HA has an active login session. I may report that separately as a new issue and close this if you think I should do that.

gdeuss commented 2 years ago

@cpainchaud I am closing this issue. The original issue was solved by your Reolink 0.51 update. However, the issue with camera integration now seems to be the Reolink firmware .764 does not work with Reolink integration 0.51

gdeuss commented 1 year ago

No, never was able to connect. The firmware it shows on my setup is currently v3.0.0.494. It tells me that is the latest but when checking the Reolink website that latest seems to be v3.1.0.764. If you want me to use a specific version, let me know, happy to try. Trouble is, that is only one of 4 different cameras I use ☹

Gustav Deuss GRD Energy, Inc. 1 Avenue of the Palms, Suite 10C San Francisco, CA 94130 O 415 891 8824 M 650 291 2569

From: klu03rl @.> Sent: Tuesday, March 8, 2022 1:45 PM To: fwestenberg/reolink_dev @.> Cc: Gustav Deuss @.>; Author @.> Subject: Re: [fwestenberg/reolink_dev] Error when adding Reolink RLC-820A (Issue #463)

did you have this issue after upgrading the firmware?

— Reply to this email directly, view it on GitHubhttps://github.com/fwestenberg/reolink_dev/issues/463#issuecomment-1062242597, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWV5XYJRINACEZ3BU4J5AXTU67C5LANCNFSM5QE5546A. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you authored the thread.Message ID: @.**@.>>