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

rtl_433 MQTT illegal discovery topic for Skylink devices (spaces?) #1487

Closed
Snuffy2 opened this issue Feb 25, 2023 · 3 comments
Closed

rtl_433 MQTT illegal discovery topic for Skylink devices (spaces?) #1487

Snuffy2 opened this issue Feb 25, 2023 · 3 comments

Comments

@Snuffy2
Copy link

Snuffy2 commented Feb 25, 2023

Describe the bug
I don't have any Skylink devices, but it appears that my neighbors may have some as they are routinely getting detected by OMG. They are giving the MQTT error in Home Assistant below. I don't know for sure, but it appears it may be from the spaces in the name. Per Home Assistant documentation they do't support spaces in the topic. Could the spaces be replaced by underscores?

Home Assistant error:
2023-02-25 17:42:32.004 WARNING (MainThread) [homeassistant.components.mqtt.discovery] Received message on illegal discovery topic 'homeassistant/binary_sensor/Skylink HA-434TL motion sensor-08000-motion/config'. The topic contains not allowed characters. For more information see https://www.home-assistant.io/docs/mqtt/discovery/#discovery-topic

Environment (please complete the following information):

  • OpenMQTTGateway version: v1.2.0
  • Hardware: Heltec WiFi LoRa 32 (V2.1)
  • Library: rtl_433

Screenshot 2023-02-25 at 5 49 51 PM

@NorthernMan54
Copy link
Collaborator

I also encountered similar concerns and this development pull request resolves them #1489

Once this is merged, you should be able to install a development build that resolves this.

@1technophile
Copy link
Owner

1technophile commented Mar 6, 2023

You can turn off auto-discovery with the following command:
https://docs.openmqttgateway.com/use/gateway.html#auto-discovery

The board will stop creating these entities

If you need to discover new devices you can turn it on again.

1technophile added a commit that referenced this issue Mar 6, 2023
revert #1365, indeed the switch may be usefull to avoid non desirable discovery of RTL_433 or BT devices  #1487
1technophile added a commit that referenced this issue Mar 6, 2023
revert #1365, indeed the switch may be usefull to avoid non desirable discovery of RTL_433 or BT devices  #1487
@1technophile
Copy link
Owner

Note that with the last version, auto discovery is now turned OFF after 30minutes
Also the switch is now visible in HA

Closing, let me know if you have any question

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

3 participants