Mqtt Broker limition of 32 character #3353
Labels
Milestone
Comments
blazoncek
added a commit
that referenced
this issue
Sep 1, 2023
- for max server name length - for max topic length Fix for #3353
Aircoookie
pushed a commit
that referenced
this issue
Sep 1, 2023
- for max server name length - for max topic length Fix for #3353
|
Why the rush to close , could at least wait to test but i guess |
|
It was closed automatically as it was linked to PR. |
|
No worries , i am just not sure how to test it which i wanted to add result here or to the PR , there is a compile flag now ? |
|
If you don't disable MQTT then add -D MQTT_MAX_SERVER_LEN=64 and if topic length is giving you griefs then -D MQTT_MAX_TOPIC_LEN=128 will help you out. |
|
Sure you mean help the guy who uses that free site site :) as i am only testing to help you out debug this I did try 54 before but not 64 or MQTT_MAX_TOPIC_LEN=128 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What happened?
As per the original post in discord ( we are only posting as asked ) we have tested with some of the free sites and Mqtt Brokers larger then 32 character will not be allowed in wled sync page
Original post :
https://discord.com/channels/473448917040758787/473449548279185408/1146929775798136842
Broker tested is HiveMq which works fine with other mqtt clients
To Reproduce Bug
Enter more then 32 char in Mqtt Broker address
Expected Behavior
More then 32 should be allowed ( the site we tested needed 51 char but could be more in the future )
Install Method
Self-Compiled
What version of WLED?
14.0.b4
Which microcontroller/board are you seeing the problem on?
ESP32
Relevant log/trace output
No response
Anything else?
No response
Code of Conduct
The text was updated successfully, but these errors were encountered: