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 integration does not support an empty password #136032

Closed
ecksun opened this issue Jan 19, 2025 · 4 comments
Closed

Reolink integration does not support an empty password #136032

ecksun opened this issue Jan 19, 2025 · 4 comments
Assignees

Comments

@ecksun
Copy link
Contributor

ecksun commented Jan 19, 2025

The problem

The default credentials of the RLC-811A camera is "admin" with an empty password, that is however not possible to enter in the UI:

Image

I noticed a comment by @starkillerOG that stated that the integration on purpose only allows a subset of "valid" passwords: #123302 (comment)

It does however not mention if empty passwords are rejected per design

If it is per design it would be nice if the error-message could indicate that ("An empty password can cause issues and is thus disallowed", "An empty password is bad practice and not allowed")

What version of Home Assistant Core has the issue?

core-2025.1.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

reolink

Link to integration documentation on our website

https://www.home-assistant.io/integrations/reolink

Additional information

I do not mind contributing a patch for this

It is also not possible to change the default password without using a proprietary program: QuantumEntangledAndy/neolink#303

@home-assistant
Copy link

Hey there @starkillerOG, mind taking a look at this issue as it has been labeled with an integration (reolink) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of reolink can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign reolink Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


reolink documentation
reolink source
(message by IssueLinks)

@starkillerOG
Copy link
Contributor

You should just go to the IP of the camera in a browser, there you will be able to set it up and provide a password.
See this section of the documentation: https://www.home-assistant.io/integrations/reolink/#1-initializing-and-configuring-camera-credentials

Before setting up the reolink homeassistant integration, you are expected to setup the camera, linking it to your network (WiFi credentials if needed) and setup the camera password.

@starkillerOG
Copy link
Contributor

Mybe in the future I will implement setting up and configuring the password for a brand new camera in the config flow of HomeAssistant. But I probably want to do that after implementing the UDP discovery protocol.
For now I first have some other features to work on.

@ecksun
Copy link
Contributor Author

ecksun commented Feb 3, 2025

You should just go to the IP of the camera in a browser, there you will be able to set it up and provide a password.
See this section of the documentation: https://www.home-assistant.io/integrations/reolink/#1-initializing-and-configuring-camera-credentials

Before setting up the reolink homeassistant integration, you are expected to setup the camera, linking it to your network (WiFi credentials if needed) and setup the camera password.

It is not possible to change the password via the web-interface if the password is the empty string, which it is the default password.

Technically that is a bug in the reolink web interface, I guess they assume one uses their app to configure the camera (I used neolink).

All that said neolink now has support to configure the password, so for the people who can't/don't want to use the proprietary reolink mobile application there is a solution available

Mybe in the future I will implement setting up and configuring the password for a brand new camera in the config flow of HomeAssistant. But I probably want to do that after implementing the UDP discovery protocol.

That would be cool but I can understand you wanting to prioritize other features :)

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

No branches or pull requests

2 participants