Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CSecurityTLS: change the variable that x509 authentication CA and CRL file from global to local #1808

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KangLin
Copy link
Contributor

@KangLin KangLin commented Aug 16, 2024

CSecurityTLS: change the variable that x509 authentication CA and CRL file from global to local.
Use different CA files for multiple connections.

@CendioOssman
Copy link
Member

I'm unsure about this approach. Feels like we're papering over a more fundamental problem. User's shouldn't have to fiddle with this setting every time they change which server they connect to.

Does GnuTLS read just a single certificate from that file? Maybe the correct usage model is to add all your trusted certificates to it, rather than having many files?

@CendioOssman
Copy link
Member

I had a look at the documentation, and it claims that you can add multiple certificates to it:

https://www.gnutls.org/manual/html_node/Core-TLS-API.html#gnutls_005fcertificate_005fset_005fx509_005ftrust_005ffile

Is that sufficient for your use case?

@KangLin KangLin marked this pull request as draft September 5, 2024 03:59
@Neustradamus
Copy link

To follow the @KangLin PR :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants