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

Feature Request: MQTT Username 64 Bytes #516

Closed
Sinned42 opened this issue Dec 22, 2022 · 4 comments
Closed

Feature Request: MQTT Username 64 Bytes #516

Sinned42 opened this issue Dec 22, 2022 · 4 comments
Assignees
Labels
enhancement New feature or request fixed dev fixed

Comments

@Sinned42
Copy link

Das mit dem SSD1306 ist schon sehr geil und echt ein Vorteil gegenüber OpenDTU. Leider fehlt mir für meinen MQTT-Broker flespi.io noch ein Byte für den Username (dort werden Tokens mit 64 Byte verwendet). Hier ist es 64 Byte - 1 Byte, also für mich leider fail. Würde mich freuen, wenn es mit den 64 Byte irgendwann klappt, dann kann ich den ESP32 mit OpenDTU abschalten.

@lumapu lumapu self-assigned this Dec 22, 2022
@lumapu lumapu added the enhancement New feature or request label Dec 22, 2022
@lumapu
Copy link
Owner

lumapu commented Dec 22, 2022

wird eingebaut - nächstes Dev-Build erlaubt dann 64 Zeichen.

lumapu added a commit that referenced this issue Dec 22, 2022
increased MQTT user, pwd and topic length to 64 characters + `\0`. (The string end `\0` reduces the available size by one) #516
@lumapu lumapu added the fixed dev fixed label Dec 22, 2022
@Sinned42
Copy link
Author

Thanks a lot, just in Time to test before the sun goes down, works perfekt with 64 byte username :)

@Sinned42
Copy link
Author

Not really important, but I just had to figure out, that that username for mqtt is only transmitted, if you put in something for the password too, which is in my case with flespi would not have been necessary. Same for radio, if you don't give latitude and longitude, the NRF didn't work, at least in versions before.

@lumapu
Copy link
Owner

lumapu commented Dec 22, 2022

thank you for figuring that out! Second one is realy important.

@lumapu lumapu closed this as completed Dec 30, 2022
@stefan123t stefan123t changed the title MQTT Username Feature Request: MQTT Username 64 Bytes Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fixed dev fixed
Projects
None yet
Development

No branches or pull requests

2 participants