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

Add support for secure websockets #419

Merged
merged 8 commits into from
Mar 28, 2025
Merged

Conversation

embhorn
Copy link
Member

@embhorn embhorn commented Mar 6, 2025

Adding secure websocket support using libwebsockets.

Fixes #37

devin-ai-integration bot and others added 4 commits March 6, 2025 17:30
…work interface

This commit adds support for secure websockets (WSS) by:
1. Modifying the libwebsocket network interface to support TLS
2. Updating the websocket example to optionally use TLS
3. Updating the build system to allow both websocket and TLS support
4. Updating documentation to reflect the new secure websocket support

Co-Authored-By: [email protected] <[email protected]>
@embhorn embhorn self-assigned this Mar 6, 2025
@embhorn embhorn force-pushed the devin_wss branch 4 times, most recently from d8b81d5 to 277d33b Compare March 7, 2025 13:57
@embhorn embhorn requested a review from philljj March 7, 2025 14:46
philljj

This comment was marked as resolved.

@embhorn
Copy link
Member Author

embhorn commented Mar 12, 2025

When using secure websockets it fails to receive large data. Not a problem with normal websockets (no TLS). Not sure if I'm doing something wrong here.

I am able to reproduce this. Bumping up the WOLFMQTT_LWS_RX_BUF_SIZE buffer size to 4Kb masks the issue.

There have been some similar issues in LWS: warmcat/libwebsockets#309

I'll keep investigating...

@embhorn embhorn requested a review from philljj March 26, 2025 20:02
@embhorn embhorn assigned philljj and unassigned embhorn Mar 28, 2025
@philljj philljj merged commit 882d035 into wolfSSL:master Mar 28, 2025
12 checks passed
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.

Support for MQTT-over-Secure-WebSockets
2 participants