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

[BUG] dots or space in ha user name break mqtt sensor discovery topic #239

Closed
hugalafutro opened this issue Sep 9, 2022 · 3 comments
Closed
Labels
bug Something isn't working

Comments

@hugalafutro
Copy link

hugalafutro commented Sep 9, 2022

Describe the bug
HA log complains about illegal mqtt discovery:

Received message on illegal discovery topic 'homeassistant/sensor/double-take/c.g.b.-spender/config'. The topic contains not allowed characters. For more information see https://www.home-assistant.io/docs/mqtt/discovery/#discovery-topic

I assume double-take pulls this name from ha (ha user is C.G.B. Spender), as I can't find the name anywhere in double-take, where the only trained face is called David.

Version of Double Take
v1.12.1-35874f3

Expected behavior
I'm not really sure, both the "unknown" and "david" sensors work ok, I just always worry about warnings in logs and people then say just ignore it it's just warning, but if it's no issue then why warn about it, but I digress.

Screenshots
n/a

Hardware

  • OS: Ubuntu 22.04.1, docker 20.10.17, docker compose v2.10.2
  • Browser Chrome

Additional context
I've got nothing.

@hugalafutro hugalafutro added the bug Something isn't working label Sep 9, 2022
@hugalafutro hugalafutro changed the title [BUG] dots in ha user name break mqtt sensor discovery topic [BUG] dots or space in ha user name break mqtt sensor discovery topic Sep 9, 2022
@jakowenko
Copy link
Owner

jakowenko commented Sep 12, 2022

Thanks for reporting this. I can look into it more. When you go to the Train page on Double Take, what names do you have in the dropdown? This error is making me think the image on the Double Take side is being recognized with the name c.g.b.-spender and then is trying to update the MQTT topic using that name.

2022-09-12 at 13 18 21@2x

@hugalafutro
Copy link
Author

hugalafutro commented Sep 13, 2022

Hi,
at the moment the only name in the dropdown is david as I was following the readme instructions and it just so happens to be my name as well. At some point during setup I might have trained and then deleted a person named C.G.B. Spender so perhaps some leftover? As at first I followed some older tutorial where I manually added the mqtt sensors, then found out they're autodiscovered now so I removed the manual and rebooted ha.

screenshots:
image

image

entities detected by HA:
image

config:

# Double Take
# Learn more at https://github.com/jakowenko/double-take/#configuration
auth: true

mqtt:
  host: !secret mq_host
  username: !secret mq_user
  password: !secret mq_pass

frigate:
  url: !secret frig_host

detectors:
  compreface:
    url: !secret compre_host
    key: !secret compre_api

detect:
  match:
    confidence: 70
    purge: 168
    min_area: 2500

telemetry: false

time:
  timezone: !secret time_zone

jakowenko added a commit that referenced this issue Oct 20, 2022
## [1.13.0-beta.3](v1.13.0-beta.2...v1.13.0-beta.3) (2022-10-20)

### Bug Fixes

* remove non alphanumeric characters from MQTT topic names ([#239](#239)) ([885d8a1](885d8a1))
jakowenko added a commit that referenced this issue Oct 21, 2022
## [1.13.0](v1.12.1...v1.13.0) (2022-10-21)

### Features

* frigate matches below an area target ([3365bc7](3365bc7))
* **mqtt & api:** total count for person, match, miss, and unknown ([#223](#223)) ([2bf4406](2bf4406))
* support for MQTT TLS ([#241](#241)) ([7f37b78](7f37b78))

### Bug Fixes

* remove non alphanumeric characters from MQTT topic names ([#239](#239)) ([885d8a1](885d8a1))
jakowenko added a commit that referenced this issue Oct 26, 2022
## [1.13.0](v1.12.1...v1.13.0) (2022-10-21)

### Features

* frigate matches below an area target ([3365bc7](3365bc7))
* **mqtt & api:** total count for person, match, miss, and unknown ([#223](#223)) ([2bf4406](2bf4406))
* support for MQTT TLS ([#241](#241)) ([7f37b78](7f37b78))

### Bug Fixes

* remove non alphanumeric characters from MQTT topic names ([#239](#239)) ([885d8a1](885d8a1))
@msolomos
Copy link

msolomos commented Jan 3, 2023

am still facing the same problem. Am currently at version 1.13.1

Received message on illegal discovery topic 'homeassistant/sensor/double-take//config'. The topic contains not allowed characters. For more information see https://www.home-assistant.io/docs/mqtt/discovery/#discovery-topic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants