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

MQTT Topic entspricht ggf. nicht üblichen Standards #44

Open
ako-3004 opened this issue Nov 21, 2021 · 6 comments
Open

MQTT Topic entspricht ggf. nicht üblichen Standards #44

ako-3004 opened this issue Nov 21, 2021 · 6 comments
Assignees

Comments

@ako-3004
Copy link

ako-3004 commented Nov 21, 2021

Im MQTT Explorer ist mir aufgefallen, dass das zugewiesene Topic nicht ganz dem normalen Standard entspricht.
Es startet mit einem "/". Alle anderen Topics die mir bisher begegnet sind starten mit einem String.
Es scheint keine Probleme dadurch zu geben, ich will das aber nicht ausschliessen.
Es wäre vermutlich sinnvoll das Topic des go-e anpassen zu können, somit vorne einen String setzen zu können.

go-e-issue
.

@0xFEEDC0DE64
Copy link
Contributor

Ich fürchte, dass wenn wir den / am Anfang nun entfernen, werden ganz viele vorhandene Integrationen nicht mehr funktionieren und das wird zu Problemen führen. / am Anfang ist ja per Standard explizit erlaubt.

@ako-3004
Copy link
Author

Ich denke wenn ihr Euch das als Task hinterlegt das Topic langfristig für den User einstellbar zu machen, dann wäre das nicht schlecht. Aktuell sehe ich auch keinen Handlungsbedarf, wollte Euch den Sachverhalt aber mal melden.

@nean-and-i
Copy link

nean-and-i commented Dec 6, 2021

Hallo,

bitte um Korrektur, bei leading slash aka zero value topics kann es zu Problemen mit mqtt libraries kommen.
Das dürfte ein Fehler sein, sowas macht keine einzige mqtt implementation.

image

Vielen Dank.

@nean-and-i
Copy link

Ich fürchte, dass wenn wir den / am Anfang nun entfernen, werden ganz viele vorhandene Integrationen nicht mehr funktionieren und das wird zu Problemen führen. / am Anfang ist ja per Standard explizit erlaubt.

leading / macht man nicht bei MQTT, man generiert dadurch nur mehr Probleme, wenn man zero value root topics erstellt.

Das ist schwerstens verpönt in der MQTT community -> Don’t use a leading slash (e.g. /mytopics ) to start your topic tree

thnx

@peterpoetzi
Copy link
Member

Wir werden mit Firmware 052.2 die Möglichkeit anbieten den Prefix für die mqtt topics setzen zu können.

Standardmäßig ist der Prefix exemplarisch mit der Seriennummer 123456:
"/go-eCharger/123456/".

Das kann man anpassen auf "go-eCharger/123456/" wenn man nur den führenden / entfernen möchte, auf "meineHomeAutomation/garageLinks/", oder einfach auf einen leeren String " ".

Der API parameter für mqtt topic prefix heißt mtp. Die Firmware 052.2 wird spätestens nächste Woche released, die App mit dem dazugehörigen Eingabefeld, kann eventuell etwas länger dauert, aber man kann den prefix ja auch über die API direkt setzen.

@nean-and-i
Copy link

Vielen Dank.

Wird man über die go-e APP (iOS/Android) darauf einen Einfluss nehmen können?

Es wäre aber sinnvoller wenn per default der Präfix entfernt wird , damit es danach dem Standard folgt -> go-eCharger/123456/

Das sollte eigentlich keine nennenswerten Probleme mit sich bringen, einmal eine bestehende integration gemäß standard umkonfigurieren (/ entfernen), es gehen ja keine Daten dadurch verloren. (z.b. breaking changes passieren ständig bei homeassistant)

@0xFEEDC0DE64 0xFEEDC0DE64 self-assigned this Oct 17, 2022
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

4 participants