Skip to content

Add support for SSL/TLS - #122

Open
Rosen01 wants to merge 1 commit into
plapointe6:masterfrom
Rosen01:master
Open

Add support for SSL/TLS#122
Rosen01 wants to merge 1 commit into
plapointe6:masterfrom
Rosen01:master

Conversation

@Rosen01

@Rosen01 Rosen01 commented Jan 21, 2023

Copy link
Copy Markdown

Answer to issue #32.

Signed-off-by: Daniel Rosengarten github@praetorians.be

Answer to issue plapointe6#32.

Signed-off-by: Daniel Rosengarten <github@praetorians.be>
@Rosen01

Rosen01 commented Jan 21, 2023

Copy link
Copy Markdown
Author

I've tested the code and it works for me.
It's retro-compatible with old code.

MQTTS (without Root CA validation -> unsecure)
EspMQTTClient client(wifi_ssid, wifi_password, mqtt_host, mqtt_login, mqtt_password, host, mqtt_port)

MQTTS (with Root CA validation)
EspMQTTClient client(wifi_ssid, wifi_password, mqtt_host, mqtt_login, mqtt_password, mqtt_ca_cert, host, mqtt_port)

MQTTS with client certificate
EspMQTTClient client(wifi_ssid, wifi_password, mqtt_host, mqtt_login, mqtt_password, mqtt_ca_cert, mqtt_client_cert, mqtt_client_key, host, mqtt_port)

@plapointe6

Copy link
Copy Markdown
Owner

Thanks a lot, I will take time to review this soon :)

@jordiyapz

Copy link
Copy Markdown

Hi, any update on the review?? @plapointe6

@yasiralijaved

Copy link
Copy Markdown

Hi, any update on this? I am trying to use this lib but can not move forward due to this limitation.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants