-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Unable to update the Topic Field - reverts to default #13167
Comments
Usually this is due to people breaking the rule about having MqttClient and Topic different, this will cause Tasmota to reset them. |
Thanks. I was not aware of this. |
I've read through the documentation "Getting Started" page on Tasmota.com and the MQTT configuration docs and could not find any mention of this. I have been using Client=Topic for some time on many devices. I would be grateful if you can point me to a doc that explains this. |
Is it possible to provide an error message instead of ignoring the new topic and restarting tasmota in case Mqttclient and topic are the same? |
I have to ask. When did this change and why? I have several devices which have the same MQTT client and topic since V6. It's now getting vital that when things change the docs need to say 'changed from Vx onwards'. On the principle stated on one of the upgrade pages 'if it ain't broke' I have devices on different versions. I upgraded a device to V10. It carried on with the same client and topic. I then moved my Mosquitto to a different device and the change on the MQTT configuration page broke several devices because it SILENTLY changed the topic setting. |
By the way, there is nothing in the MQTT specification that requires the ClientID and Topic to be different. The only requirement is that each device must have a unique ClientID. Moreover I have searched the release notes and have been unable to find a notification as to when this changed. It HAS changed because I still have devices which I have not migrated to my new Mosquitto server with identical ClientID and topics. |
The coding in Tasmota checking the sameness is not 100%, it is not impossible to sneak through a configuration likely to suddenly cause you trouble later. |
I've been able to track back this test to 6.2.1.14 On Oct 10th 2018 While the MQTT standard do not enforce that, it may also be a limitation of the PubSubClient library that we are using. Personnaly I consider it is a good habit to have the ClientD tighted to the physical layer (the hardware, based on MAC address) and the Topic tight to the logical layer (what it is used for). |
PROBLEM DESCRIPTION
A clear and concise description of what the problem is.
I a unable to update the Topic Field on Versions 9.5.0.0 and 9.5.0.8
The field is reset back to default I am trying to set topic as P81_BenchLamp or P1-BenchLamp.
Changing the topic to sexybeast DOES WORK. But that DOESN'T work for me.
REQUESTED INFORMATION
Make sure your have performed every step and checked the applicable boxes before submitting your issue. Thank you!
Backlog Template; Module; GPIO 255
:Backlog Rule1; Rule2; Rule3
:Status 0
:weblog
to 4 and then, when you experience your issue, provide the output of the Console log:TO REPRODUCE
Steps to reproduce the behavior:
update the MQTT Topic field to something with an underscore, or a dash and caps
the result is field reset to default
EXPECTED BEHAVIOUR
A clear and concise description of what you expected to happen.
I expected to be able to use the device name as the topic. If the device name is legal then it should also be fine for topic. I was using P81_BenchLamp as topic. It conveys to me that it is a Plug Module at 192.168.1.81 with a friendly name.
tasmota_2a4d13 is not useful when configuring subscriptions.
SCREENSHOTS
If applicable, add screenshots to help explain your problem.
ADDITIONAL CONTEXT
Add any other context about the problem here.
I am using this in Homeseer as well as Home Assistant
Thanks for the great work ! Even though I am bitchin I am grateful for your talents.
(Please, remember to close the issue when the problem has been addressed)
The text was updated successfully, but these errors were encountered: