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

Reolink Home Assistant integrations can't add NVR RLN36 Firmware Version v3.3.0.282_23103105 #50

Closed
juslex opened this issue Dec 9, 2023 · 22 comments

Comments

@juslex
Copy link

juslex commented Dec 9, 2023

Describe the bug
It is not possible to add the NVR RLN36 to the home assistant with firmware version v3.3.0.282_23103105

To Reproduce
When trying to enter the username and password and the ip where the nvr is located I receive the invalid authentication error message because the user and password are correct

Expected behavior
Before updating to firmware v3.3.0.282_23103105 I could normally add the nvr through integration

Screenshots

Environment:
Please provide useful information about your environment, like:
Model RLN36
Build No. build 2310315
Hardware No. N5MB01
Config Version v3.0.0.0
Firmware Version v3.3.0.282_23103105

Additional context
I believe it is something that has changed with the new version that prevents the integration from authenticating. Both HTPP and HTPPS are active in the nvr configuration. I tried to log in both using HTPP and HTPPS and the bug persists.

@starkillerOG
Copy link
Owner

Please enable debug logging and post the log.txt here when you tried to setup the integration.

I need to see the communication with the device.

@starkillerOG
Copy link
Owner

If you go to the IP of the NVR using a webbrowser, are you able to login using the username and password you are using in Home Assistant?

@starkillerOG
Copy link
Owner

To enable debug loggin see here: https://www.home-assistant.io/docs/configuration/troubleshooting/#enabling-debug-logging

Or add the following to your configuration.yaml:

logger:
  default: warn
  logs:
    reolink_aio: debug
    reolink: debug
    homeassistant.components.reolink: debug

@juslex
Copy link
Author

juslex commented Dec 9, 2023

If you go to the IP of the NVR using a webbrowser, are you able to login using the username and password you are using in Home Assistant?

Yes, I did this test before opening this issue.

@juslex
Copy link
Author

juslex commented Dec 10, 2023

reolink.log
Follow the log as directed

@starkillerOG
Copy link
Owner

@juslex thanks for the debug log.
The problem originates from the camera connected to channel 17 of the NVR.
That gives all kinds of errors such as get config failed, set config failed, rcv failed and login failed.

What type of camera is connected to channel 17?
In the Reolink app/windows client, what is shown for the camera connected to channel 17?
Is it perhaps a camera not directly connected to the NVR that does not have the password of that camera configured correctly and showing a login failed in the reolink windows client?

@starkillerOG
Copy link
Owner

@juslex I tried to reproduce the issues on my RLN8 NVR by connecting a camera over LAN and then changing the password of that camera. However that just emidiatly showed that camera as offline, so that could not explain what you are seeing...

Maybe the camera on channel 17 is a wifi camera with a really bad connection???

However I am not running the same firmware and hardware NVR, so I really need you to do some testing if you want to get this resolved.

@juslex
Copy link
Author

juslex commented Dec 11, 2023

I'm checking here and I'll give you a Feedback

@juslex
Copy link
Author

juslex commented Dec 11, 2023

Updating. I saw that channel 17 had an invalid camera setting, I removed it and managed to make it work. You can close the issue, sorry for the inconvenience.

@starkillerOG
Copy link
Owner

Could you eleborate a bit more?
What model was the channel 17 camera?
Which camera setting was invalid?

I would like to see if I can detect such types of issues and make a better error message/repairs issue such that users now what to do to resolve it.

@juslex
Copy link
Author

juslex commented Dec 13, 2023

Could you eleborate a bit more? What model was the channel 17 camera? Which camera setting was invalid?

I would like to see if I can detect such types of issues and make a better error message/repairs issue such that users now what to do to resolve it.

Channel 17 was a Reolink doorbell Wifi. But the ip that was pointed out there was her first, after I changed the ip and put it on another channel and the configuration of channel 17 was wrong without me knowing.

@starkillerOG
Copy link
Owner

@juslex thank you for clearifying.
Weired that the RLN36 still showed the camera online....
Maybe because at that original IP there was some other device that was responding in some way to the NVR or something.
Sounds like pretty rare...

@juslex
Copy link
Author

juslex commented Jan 24, 2024

Hello @starkillerOG I'm experiencing an error again when authenticating the integration with my nvr, here is the log:
home-assistant.log

@starkillerOG
Copy link
Owner

@juslex thanks for reporting this.
This time channel 16 is giving issues:
Again this channel is called "NVT" and the "typeInfo" : "IPC".

I would like to get rid off this issue in the software, so please do not remove this channel 16 yet!

Did you rename channel 16 to "NVT" or did it do that on its own?
Is there any camera connected to this channel 16?

Again, please do not remove this channel yet, maybe some additional tests are needed....

@starkillerOG
Copy link
Owner

Could you give me a screenshot on how this channel 16 looks like in the reolink desktop client and phone app?

@starkillerOG
Copy link
Owner

@juslex did you rename channel 16 to "NVT" yourself or did it do that on its own?

@juslex
Copy link
Author

juslex commented Feb 6, 2024

Hello @starkillerOG this channel was from a non-Reolink camera in the ONVIF standard. It's funny that the first few times it worked, however, the integration started to give a problem. I ended up removing it from the nvr and it worked again. I suggest as an improvement in the integration, the possibility of reporting the type of error, currently, we only receive the information in the dialog box that the login was unsuccessful.

@starkillerOG
Copy link
Owner

@juslex I made a possible fix here: #59
But that relies on the camera name beeing "NVT", in that way I can ignore these non-Reolink camera's.
But in order to do that I need to know if they will always be called "NVT" or that you named it yourself like that.

@juslex
Copy link
Author

juslex commented Feb 6, 2024

The name was automatically assigned by the NVR. I even tried to change it, but without success.

@starkillerOG
Copy link
Owner

@juslex thanks for the info!

@starkillerOG
Copy link
Owner

@juslex the next HA 2024.3.2 patch release will include this PR home-assistant/core#110959 that will ensure these non-reolink camera's will be ignored in the HomeAssistant Reolink integration.

So the problem schould now be solved indefenitly.

If you appreciate the reolink integration and want to support its development, please consider sponsering the upstream library or purchase Reolink products through this affiliate link.

@juslex
Copy link
Author

juslex commented Feb 19, 2024

Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants