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

DIYHue overwrites gateway in config file #943

Closed
sosheskaz opened this issue Oct 25, 2023 · 3 comments · Fixed by #992
Closed

DIYHue overwrites gateway in config file #943

sosheskaz opened this issue Oct 25, 2023 · 3 comments · Fixed by #992

Comments

@sosheskaz
Copy link
Contributor

Issue does not already exist?

I have searched and found no existing issue

Select Environment

Docker

Home Assistant related?

No

Description

When starting up, diyhue overwites any previous entries to gateway in config.yml, replacing it with a default.

Based on behavior, I suspect it is defaulting to the first (zero-indexed) IP address in the /24 subnet of the host IP.

This prevents configuration of a non-default gateway.

Errorlog:

N/A not an error.

Steps to reproduce

Install as a home assistant container via Hassio.

  1. Read config file. Note gateway value.
  2. Change value of gateway. Note value.
  3. Restart diyhue.
  4. Read config file again. Note gateway value. It is now the same value as step 1.

Please enter your operating system details here

HAOS. Linux 6.1.56 x86_64

What DiyHue version(branch) are you using?

master (latest)

@sosheskaz
Copy link
Contributor Author

yaml_config["config"]["gateway"] = ip_pieces[0] + "." + ip_pieces[1] + "." + ip_pieces[2] + ".1"

Yeah, not a reliable way to configure gateway

@sosheskaz
Copy link
Contributor Author

I believe this is causing linking to fail for me, due to the configured network settings being incorrect.

@cibrenvenuti
Copy link

Same issue here

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

Successfully merging a pull request may close this issue.

2 participants