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 Password length only 31 chars possible #182

Closed
Chrischi- opened this issue Aug 19, 2022 · 6 comments
Closed

MQTT Password length only 31 chars possible #182

Chrischi- opened this issue Aug 19, 2022 · 6 comments
Labels
bug Something isn't working fixed dev fixed

Comments

@Chrischi-
Copy link

Chrischi- commented Aug 19, 2022

MQTT connect with user + password not possible

Mosquitto MQTT Broker Log:
1660944902: New connection from 172.16.0.29:55936 on port 1883.
1660944902: Client disconnected, not authorised.

MQTT Password length only 31 chars possible
Everything after 31 characters is truncated and not saved.

I have tested all versions from 5.5 to 5.14 without success. I had not yet time to look for the issue in the source.

@stefan123t
Copy link
Collaborator

stefan123t commented Aug 19, 2022

Most likely a one off limitation as we are writing only 32 chars into EEPROM we may miss the \0 String ending. We had this before for WiFi Password too see #11

@stefan123t stefan123t added bug Something isn't working good first issue Good for newcomers labels Aug 19, 2022
@Chrischi-
Copy link
Author

Chrischi- commented Aug 20, 2022

Brief additional info:
Login is not possible even with shorter password.
However, a MQTT listener without authentication works without problems.

Tested up to 5.15

@aschiffler
Copy link
Contributor

Will leave this open for the next release >0.5.16

@Chrischi-
Copy link
Author

Maybe it was me, but with 0.5.16 I have no more problems logging in.

However, the issue with the password length still exists.

@Chrischi- Chrischi- changed the title MQTT connect with user + password not possible; MQTT Password length only 31 chars possible MQTT Password length only 31 chars possible Sep 4, 2022
@aschiffler
Copy link
Contributor

Close this bc. password length of 31 characters shall be sufficient.

@Chrischi-
Copy link
Author

@aschiffler
OK, then I would also include the maximum password length somewhere in the documentation. Otherwise, this issue will surely repeat itself at some point.

@stefan123t stefan123t added fixed dev fixed and removed good first issue Good for newcomers labels Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixed dev fixed
Projects
None yet
Development

No branches or pull requests

3 participants