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 TLS (Fingerprint) #2625

Closed
Joci78 opened this issue Sep 23, 2023 · 6 comments
Closed

MQTT TLS (Fingerprint) #2625

Joci78 opened this issue Sep 23, 2023 · 6 comments

Comments

@Joci78
Copy link

Joci78 commented Sep 23, 2023

The Feature

Dear Developer,

It would be good if you know about MQTT TLS communication. Tasmota's solution works perfectly (fingerprint). I could imagine something similar here.

Best regards,
Joc

@Joci78 Joci78 added the enhancement New feature or request label Sep 23, 2023
@caco3
Copy link
Collaborator

caco3 commented Sep 27, 2023

Please add more information, eg. a link to the Tasmota doc/spec/implementation.

@Joci78
Copy link
Author

Joci78 commented Sep 28, 2023

Tasmota documentation page:
https://tasmota.github.io/docs/TLS/

@LordGuilly
Copy link
Contributor

I submitted a PR to use certificates with MQTT, I tested with AWS, it uses the ESP libraries

@caco3
Copy link
Collaborator

caco3 commented Oct 20, 2023

Nice work! I will have a look on #2651 end of next week as I am on holiday now.

Since you added new parameters, we also have to update the documentation.
Can you create the pages CACert.md, ClientCert.md and ClientKey.md in https://github.com/jomjol/AI-on-the-edge-device-docs/tree/main/param-docs/parameter-pages/MQTT? They then will be used for the tooltips on the UI.

Orient yourself on the other files. Also i think this should be expert parameters.

@caco3 caco3 self-assigned this Oct 22, 2023
@bucklevision
Copy link

Super keen to have TLS support, my broker wants to go 8883 only soon and I'm keen not to lose functionality!

@caco3
Copy link
Collaborator

caco3 commented Nov 7, 2023

I could not test it myself but I merged it now.

Please test it and give a feedback here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants