-
Notifications
You must be signed in to change notification settings - Fork 58
Open
Labels
bugSomething isn't workingSomething isn't workingconfirmedWe know what should be done to resolve the issueWe know what should be done to resolve the issue
Description
Description
The configured port of the container is not respected for the homeassistant notification.
This result in a wrong "enable pairing mode" link in the Home Assistant notification.
I guess the issue is that the port is not added to the "ip_address" in this code line:
core/emulated_hue/controllers/config.py
Line 391 in 96081a3
| url = f"http://{self.ip_addr}/link/{self._link_mode_discovery_key}" |
Expected behavior
The notification in homeassistant has a link which is also working for non standard ports (currently 80 is "hard-coded")
Reproduction steps
- Run the container on a different port than 80
- Click the notification link in Home Assistant
- See the links going to port 80 and thus not working
Relevant log output
N/AOperating system
Other
Install method
Docker CLI
Version
0.3.0
Dev Version
No response
Any other information that may be helpful
No response
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingconfirmedWe know what should be done to resolve the issueWe know what should be done to resolve the issue