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

LWT_TOPIC missing #144

Open
Bolukan opened this issue Jun 18, 2023 · 1 comment
Open

LWT_TOPIC missing #144

Bolukan opened this issue Jun 18, 2023 · 1 comment

Comments

@Bolukan
Copy link

Bolukan commented Jun 18, 2023

Describe the bug
Setting the environmental variable LWT_TOPIC does not change the used LWT topic.

Could you add the LWT_TOPIC which default is home/TheengsGateway/LWT to the supported environment variables as I prefer a different topic.

Thx

To Reproduce
Add LWT_TOPIC as variable to the composer file:

Only relevent line added:
environment:
LWT_TOPIC: ble/theengs/LWT

Restart container and check the MQTT output: nothing changes.

Expected behavior
The LWT_TOPIC can be set.

Additional context
The docker file is not processing the variable and needs an update to include LWT_TOPIC as possible parameter.

@Bolukan
Copy link
Author

Bolukan commented Jun 18, 2023

I wrote the needed change: theengs/gateway-docker#6

I did some checks to comply to the quality of the current code:
Like other mqtt topics it is a free text fields and no checks are to be performed, only add it to the config file.
Second, I looked into the config file of a running instance to add the line to the same place.

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

No branches or pull requests

1 participant