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

libvncclient: improve compatibility with GnuTLS 3.8 #630

Merged
merged 1 commit into from
Sep 23, 2024

Conversation

yan12125
Copy link
Contributor

Since GnuTLS 3.8, TTLS-SRP is disabled by default at compile time [1]. As a result, connecting to a server via TLS fails with:

Warning: Failed to set TLS priority: The request is invalid. (+SRP).

TLS handshake failed: No or insufficient priorities were set.

On the other hand, TLS-SRP is apparently unused in libvncclient. It seems TLS-SRP requires providing serveral parameters, including a username and a password, via specific GnuTLS API [2], which I didn't find in the codebase.

[1] https://gitlab.com/gnutls/gnutls/-/merge_requests/1681
[2] https://www.gnutls.org/manual/html_node/Authentication-using-SRP.html

Since GnuTLS 3.8, TTLS-SRP is disabled by default at compile time [1].
As a result, connecting to a server via TLS fails with:

> Warning: Failed to set TLS priority: The request is invalid. (+SRP).

> TLS handshake failed: No or insufficient priorities were set.

On the other hand, TLS-SRP is apparently unused in libvncclient. It
seems TLS-SRP requires providing serveral parameters, including a
username and a password, via specific GnuTLS API [2], which I didn't
find in the codebase.

[1] https://gitlab.com/gnutls/gnutls/-/merge_requests/1681
[2] https://www.gnutls.org/manual/html_node/Authentication-using-SRP.html
@bk138 bk138 merged commit 2a23c44 into LibVNC:master Sep 23, 2024
12 checks passed
@bk138
Copy link
Member

bk138 commented Sep 23, 2024

Thanks!

@yan12125 yan12125 deleted the fix-for-gnutls-3.8 branch September 23, 2024 13:10
@yan12125
Copy link
Contributor Author

Whoa, that's blazingly fast! Thank you very much.

@bk138
Copy link
Member

bk138 commented Sep 23, 2024

That was just a coincidence with me being online and the change being very small ;-)

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.

2 participants