-
-
Notifications
You must be signed in to change notification settings - Fork 33.8k
Can't change the port number for netgear integration #57196
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
Comments
Hey there @hacf-fr, @Quentame, @starkillerOG, mind taking a look at this issue as it has been labeled with an integration ( netgear documentation |
@yiweiyu32 if the device is automatically discovered you can't change the port, but it schould be the correct port actually. |
@starkillerOG Agree on the 2nd part. Was able to configure it manually. In my case the automatically discovered default port (5000) is not correct since I'm using a Orbi router (port 80) and that's what this issue is about. |
I included a debug log for the SSDP discovery information in this PR: #57226 |
@starkillerOG |
@yiweiyu32 so this basically only shows 5000 as the port, but no where does it refrence the correct port: 80. |
|
well I don't see the correct port 80 in here as well. |
Yea, I can't find any mentioning of port 80. Just so you're aware this (Orbi's having a different default port) is pointed out in this page https://www.home-assistant.io/integrations/netgear/ but I'm not sure if that list is exhaustive. |
Another way to look at this is that since port 80 is the default http port, if there's no mention of port 5000 in some specific places (in some http urls) maybe we should set the default port to 80 |
Yes I am aware, I re-wrote that whole page :) |
All other models exept the orbi range use port 5000. I think using the model from SSDP information and matching to known orbi models here core/homeassistant/components/netgear/const.py Lines 14 to 31 in 7e34860
is the best option. |
Awesome. Will help test it out once the cha 8000 nge's been released. Thanks! |
@yiweiyu32 I think this PR will fix the auto-discovery: #57432 |
@yiweiyu32 the fix will be in HomeAssistant 2021.10.4, could you test that once it comes out in a few days and see if auto-discovery now works withouth specifying the port? |
@starkillerOG upgraded 2021.10.4, restarted home assistant and it's still showing that the auto discovered port is 5000. |
@yiweiyu32 are you sure you are on 2021.10.4? In the discovery debug log you posted here #57196 (comment), not everything was included. Specifically I match the "modelNumber" to the known Orbi models. Could you provide the "modelNumber" as listed in the discovery debug log? |
@starkillerOG Checked the hass version, removed the existing integration, restarted hass and I'm still seeing the wrong port (5000). |
Alright, so normally the |
This PR will also check the |
Thanks! Will retest it. What would the fix version be? @starkillerOG |
@yiweiyu32 it will be in 2021.10.7 |
Tested this in core-2021.11.4. The integration can now discover the correct port for my orbit router. |
The problem
With 2021.10, netgear migration has moved from yaml to UI. However the port number defaults to 5000 and there's no option to set it to a different port to account for products like orbi routers.
What is version of Home Assistant Core has the issue?
core-2021.10.0
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
Netgear
Link to integration documentation on our website
No response
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response
The text was updated successfully, but these errors were encountered: