-
Notifications
You must be signed in to change notification settings - Fork 57
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
TLS certificate support #29
Comments
I'm not actively using it myself anymore/rightnow, but I'll do my best to merge things. There's already support for TLS-PSK, but I hadn't needed cert based stuff, I'm sure someone would find it useful :) In general, SSL is virtually zero overhead until your packet sizes get "large" as the main cost is only at connection time. |
+1 for @swails 's suggestion. I am personally working on TLS and self-signed --cafile based mosquitto server. So, I feel that would be a great addition to test my service. |
Please see my earlier comment. I'm sure it would be a useful addition, and I' try and merge any PR that adds this. However, I'm not working on it myself. |
Hi there |
Please read the earlier comments. |
As per the previous comments, it does not talk about how to use this tool with TLS. |
As mentioned, every time, #29 (comment) there is no support for certificate based authentication. A Pull request would be reviewed. TLS-PSK is supported. I've locked this because, as before, I'm not working on this, and saying the same thing every time is tedious. |
Hi,
I wasn't completely sure that this project was still active, but recent activity on #28 suggests that's not necessarily true. I'm working on SSL/TLS support in my own fork (with support for anonymous encryption with self-signed certs). Would that be a welcome addition as a PR here? I believe encryption adds a bit of overhead to the bandwidth consumption, so I wanted the stress testing done with that turned on (since it will be in production for my needs).
I'm not tackling client certs since that's not part of my use-case, but it should presumably be straightforward to make the addition if someone wanted it.
Thanks.
The text was updated successfully, but these errors were encountered: