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

SSL/TLS support for MQTT #505

Closed
dennissiemensma opened this issue Jul 22, 2018 · 6 comments
Closed

SSL/TLS support for MQTT #505

dennissiemensma opened this issue Jul 22, 2018 · 6 comments
Milestone

Comments

@dennissiemensma
Copy link
Member

From #417:

Kunnen we ook SSL/TLS ondersteuning toevoegen aan MQTT? Security wordt steeds belangrijker ;-) (al mijn andere devices doen all MQTT met TLS v1.1/1.2)

@dennissiemensma
Copy link
Member Author

I've managed to make this work, I'll add three choices:

  • Insecure (as is)
  • Secure, but no certificate validation (for self-signed)
  • Secure, with validation (most secure, but requires a 'real' certificate)

Connecting insecure to MQTTS results in (mosquitto logs):

OpenSSL Error: error:140760FC:SSL routines:SSL23_GET_CLIENT_HELLO:unknown protocol

Connecting to a self-signed certificate, with the most strict validation will result in:

OpenSSL Error: error:14094418:SSL routines:ssl3_read_bytes:tlsv1 alert unknown ca
OpenSSL Error: error:140940E5:SSL routines:ssl3_read_bytes:ssl handshake failure

dennissiemensma added a commit that referenced this issue Jul 30, 2018
dennissiemensma added a commit that referenced this issue Jul 30, 2018
@ualex73
Copy link

ualex73 commented Jul 31, 2018

Vluchtig getest met no certificate validation, maar toen stopte MQTT met werken (kreeg oneindig veel errors in de mosquitto log) ... Ik ga vanavond verder testen, maar eerst ffies mqtt fixen, want deze weigert om op te starten (nadat ik hem op none-secure heb gezet).

@ualex73
Copy link

ualex73 commented Jul 31, 2018

Aha!

Mijn MQTT username/password waren weer default geworden (upgrade script?), vandaar dat de connectie mislukt.

@ualex73
Copy link

ualex73 commented Jul 31, 2018

Ik heb het nogmaals geprobeerd en nu werkt het wel goed, mijn password manager paste de username/password aan :-( Ik kan alleen niet de valid certificate testen, want ik heb alleen self signed certificaten.

Dus de MQTT/MQTTS support is approved 👍

@dennissiemensma
Copy link
Member Author

Top, fijn om te horen dat het uiteindelijk gelukt is en voor jou werkt. Bedankt voor het proberen! Ik zal dit dan terugmergen voor de volgende release. Tot die tijd kun je het blijven draaien.

dennissiemensma added a commit that referenced this issue Jul 31, 2018
@dennissiemensma
Copy link
Member Author

@ualex73 je kunt nu wisselen naar de development branch voor nu. Daar zit zowel de refactoring als MQTTS in.

sudo su - dsmr
git fetch
git checkout -b development origin/development
./deploy.sh

@dennissiemensma dennissiemensma added this to the 1.23 milestone Jul 31, 2018
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

No branches or pull requests

2 participants