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

Home Assistant auto discovery broken due to bad topic subscription #2554

Closed
mill1000 opened this issue Jul 6, 2023 · 4 comments · Fixed by #2593
Closed

Home Assistant auto discovery broken due to bad topic subscription #2554

mill1000 opened this issue Jul 6, 2023 · 4 comments · Fixed by #2593

Comments

@mill1000
Copy link
Contributor

mill1000 commented Jul 6, 2023

Looks like the Home Assistant auto discovery (rtl_433_mqtt_hass.py) is broken. Possibly introduced by #2289.

The topic subscriptions that are created for the sensors are missing the proper topic prefix so no values are received.
New subscription
image

Previous subscription
image

rtl_433 command

docker run --device /dev/bus/usb/001/007 hertzg/rtl_433:latest -C si -F "mqtt://10.100.1.6:1883,user=,pass=" -F log -F kv  -R 162 -M level
@mill1000
Copy link
Contributor Author

mill1000 commented Jul 6, 2023

Here's a working config using the current main.

rtl_433_mqtt_hass.py -d -T rtl_433/devices[/type][/model][/subtype][/channel][/id]

rtl_433

rtl_433 -C si -F "mqtt://10.100.1.6:1883,devices=rtl_433/devices[/type][/model][/subtype][/channel][/id],events=rtl_433/test/events,states=rtl_433/test/states" -F log -F kv -M level

Not ideal since I had to put an extra level in the events and states topics

@zuckschwerdt
Copy link
Collaborator

@unverbraucht can you check this please?

@unverbraucht
Copy link
Contributor

Looking into it.

@unverbraucht
Copy link
Contributor

I could reproduce the issue, looking into it now.

unverbraucht added a commit to unverbraucht/rtl_433 that referenced this issue Aug 11, 2023
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

Successfully merging a pull request may close this issue.

3 participants