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 dont accept Port setting for connection #91109

Closed
kedar-1 opened this issue Apr 9, 2023 · 4 comments · Fixed by #91137
Closed

Reolink dont accept Port setting for connection #91109

kedar-1 opened this issue Apr 9, 2023 · 4 comments · Fixed by #91137

Comments

@kedar-1
Copy link

kedar-1 commented Apr 9, 2023

The problem

Reolink Integration don't accept port except 80.
I have setup a port 8081 for my camera.
I'm not able to to add my camera to HA.

No matter of what info was in the Port field only 80 was sent to the camera login.

Client connector error during login of host 192.168.1.101:80: Cannot connect to host 192.168.1.101:80 ssl:default [Connect call failed ('192.168.1.101', 80)]

What version of Home Assistant Core has the issue?

core-2023.3.6

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Supervised

Integration causing the issue

Reolink

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

home-assistant bot commented Apr 9, 2023

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.

(message by CodeOwnersMention)


reolink documentation
reolink source
(message by IssueLinks)

@starkillerOG
Copy link
Contributor

I just tested on my setup, at first the port field will not be shown (because 95% of users will not need it), so it will always give you a connection error.
Then the port field will pop up and you can put in your custom port.
However you also need to fill in if you use HTTPS or not, so if you don't you will have to toggle it on and then off.

I now see this is not nice, so will make sure this is changed so you only need to put in the port.

But for you, just toggle the use HTTPS on and then back off while filling in your port.

@starkillerOG
Copy link
Contributor

@kedar-1 This PR #91137 will make sure you do not have to toggle on/off the USE_HTTPS checkbox anymore.

@starkillerOG
Copy link
Contributor

If you appreciate the reolink integration and want to support its development, please consider sponsering the upstream library.

@github-actions github-actions bot locked and limited conversation to collaborators May 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants