Skip to content

Port not set for HomeAssistant notification #513

@eloo

Description

@eloo

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:

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/A

Operating 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

No one assigned

    Labels

    bugSomething isn't workingconfirmedWe know what should be done to resolve the issue

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions