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

Behavior of tls_version #2110

Closed
vertecx opened this issue Mar 3, 2021 · 1 comment
Closed

Behavior of tls_version #2110

vertecx opened this issue Mar 3, 2021 · 1 comment
Milestone

Comments

@vertecx
Copy link

vertecx commented Mar 3, 2021

I still seem to have the old behavior of the tls_version option specifying the only supported TLS version, instead of the minimum version, when running the broker version 2.0.8 on Windows. I have not tried on other platforms.

I have tried connecting with different TLS versions using openssl s_client -tls1_X -connect localhost:8883 and I can only connect using the exact version specified in tls_version. Unless not specifying tls_version at all, then I can use TLS 1.1, 1.2 and 1.3.

Could it be that the changes made to lib/net_mosq.c in commit abbeb44 must also be made in src/net.c around line 332? The TLS version code there seems very similar to the old net_mosq code and seems to match the behavior I am seeing when running the broker.

ralight added a commit that referenced this issue Mar 3, 2021
It was setting the exact TLS version to use, not the minimium TLS
version to use.

Closes #2110. Thanks to Petter Jönsson.
@ralight
Copy link
Contributor

ralight commented Mar 3, 2021

Thank you, well spotted. That should have been changed at the same time as the client library, and I honestly thought I had done it. It's fixed now and will be part of the next bug fix release.

@ralight ralight added this to the 2.0.9 milestone Mar 3, 2021
@ralight ralight closed this as completed Mar 11, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants