roflcoopter / viseron

Self-hosted, local only NVR and AI Computer Vision software. With features such as object detection, motion detection, face recognition and more, it gives you the power to keep an eye on your home, office or any other place you want to monitor.
MIT License
1.66k stars 171 forks source link

mqtt: client_id vs. base topic #595

Open hessijames79 opened 1 year ago

hessijames79 commented 1 year ago

According to the documentation, the configuration options for the mqtt client_id and the mqtt base topic are the same.

However, choosing a base topic such as site1/general/viseron would result in a client_id containing a slash which MAY be allowed by the server according to the specification but it is no MUST.

I suggest to add a new configuration option such as base_topic that could default to the value of client_id for backwards compatibility.

Patrick

roflcoopter commented 1 year ago

Seems like a good idea 👍