-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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 with Socket #497
Comments
|
That looks fine. Could you try running |
|
It's looks fine. Which socket client you are using for testing ? My Objective: Read and validate Username from Client Certificate. |
|
I have changed the read_handle_server function mqtt3_handle_connect, Line 330. I'm reading SSL context and copy CN into context->username. It's works for normal Port. How can I read the data from client certificate when WEB Sockets is used? |
|
Oh right, so a different question entirely :) This requires a bit of work with libwebsockets to fix, then you should be able to use |
|
Thank you for your reply. I get it. It is much work. Can you help me write a rough plan anyway? Anyway: what websocket client will you recommend? |
|
It's not actually that much work, but needs thinking to ensure nothing gets broken. You need to ensure that the listener->ssl_ctx is correct for the websockets listeners and also websockets clients have a valid context->ssl. These will almost certainly provided by libwebsockets so we shouldn't free them ourselves. I'd use the Paho Python client for websockets, but it depends entirely on what is appropriate for your situation. |
|
Closing as this is fixed by 7943072 which will be included in next release. Feel free to reopen if the issue persist. |
I would like to use secure websocket connection. Is this implemented? Have you a sample configuration for me? Which test client can I use?
My Configuration:
Mosquitto 1.4.11, OS: Fedora / Ubuntu (inside Docker).
Client. NodeJS MQTT
It works without encryption or at normal port. With SSL encryption not.
Debug says SSL Context is null. Many thanks,
The text was updated successfully, but these errors were encountered: