search

goecharger / go-eCharger-API-v2

New API specification for V3 and V4 go-eCharger
90 stars 21 forks source link

MQTT Topic beginnt mit / #84

Open digidax opened 2 years ago

digidax commented 2 years ago

Wäre es möglich, dass die MQTT Topic NICHT mit / beginnt, sondern direkt mit go-eCharger/123456

Ist im MQTT Explorer schöner go-eCharger gleich in der Root Topic zu sehen anstatt ein leeres Feld: grafik grafik

0xFEEDC0DE64 commented 2 years ago

als kurzfristigen work-around: api key "mtp", type string, "mqtt topic prefix, set to null to reset to default again"

es kann sein, dass nach ändern des prefixes ein stoppen/starten oder ganzer reboot notwendig ist, es ist noch nicht komplett durchgetestet

copacetic82 commented 1 year ago

Hat sich dieses Verhalten in der neusten version 054.6 Beta geädnert, ohne es in den release notes erwähnt wurde? Bei mir waren die topics nach dem upgrade auf einer einer anderen ebene, und diverse sensoren funktionierten dadurch nicht. Hab vorerst ein rollback auf 054.3 gemacht. Wird das auch in die nächste non-beta version übernommen?

Siehe vergleich: image

arboeh commented 1 year ago

Kann ich bestätigen und ich habe mir in den letzten Stunden nen Wolf gesucht, weil ich heute eine neue Instanz aufgesetzt habe... :-|

copacetic82 commented 1 year ago

Kann ich bestätigen und ich habe mir in den letzten Stunden nen Wolf gesucht, weil ich heute eine neue Instanz aufgesetzt habe... :-|

Ging mir ähnlich, konnte es aber zum glück recht schnell eingrenzen. Wäre cool gewesen wenns in den release notes stehen würde :/

syssi commented 1 year ago

Same here:

-/go-eCharger/072246/fhz 49.754
+go-eCharger/072246/fhz 49.754
copacetic82 commented 1 year ago

fixed in 054.7 Beta

digidax commented 1 year ago

The forward slash (/) is used to separate each level within a topic tree. Focus is the separation not the beginning: http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc398718106

MQTT Best practices Never use a leading forward slash

A leading forward slash is permitted in MQTT. For example, /myhome/groundfloor/livingroom. However, the leading forward slash introduces an unnecessary topic level with a zero character at the front. The zero does not provide any benefit and often leads to confusion. https://www.hivemq.com/blog/mqtt-essentials-part-5-mqtt-topics-best-practices/

fu-zhou commented 1 year ago

in der 54.7 hat sich irgendwie auch verändert, wie der go-e charger seine Datenpunkte über MQTT veröffentlicht - s. https://github.com/goecharger/go-eCharger-API-v2/issues/89#issuecomment-1272274910

0xFEEDC0DE64 commented 1 year ago

ich wollte eigentlich nur für neue mqtt-installationen endlich den / zu beginn los werden, in der ersten beta hat's aber den default auch für bestehende mqtt-user mit verdreht, mit dem nächsten beta update sollte es jetzt so sein, dass wenn mqtt schon aufgedreht war, er den führenden / behält und für neue mqtt user per default nur ohne / vorschlägt und man kann manuell in der app den mqtt topic prefix selber nach wunsch verändern.