added options for a CA bundle

krylovsk / mqtt-benchmark

MQTT broker benchmarking tool

Apache License 2.0

293 stars 81 forks source link

added options for a CA bundle #25

Closed scottfp closed 2 years ago

scottfp commented 3 years ago

Feedback welcome. Thanks for the tool 🙂

scottfp commented 3 years ago

Two more things: 1) I removed the error when using broker CA without a client cert/key. This is a legitimate use case when connecting over SSL, but without cert-based authentication. 2) I added a flag for toggling TLS verify. It's defaulting to false now, with the flag enabling it. Please let me know if you want to keep it as it currently is, or if you feel this should be a separate PR.

aparcar commented 2 years ago

Will this be fixed?

krylovsk commented 2 years ago

@aparcar merged

aparcar commented 2 years ago

That was quick, thanks.

krylovsk commented 2 years ago

please pull latest main, I've fixed the comment above and renamed the flag to -broker-ca-cert.