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

openssl 3 #6

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

openssl 3 #6

wants to merge 1 commit into from

Conversation

esegece-com
Copy link
Collaborator

Update the OpenSSL IOHandler to support the OpenSSL 3 libraries. Changes done:

IdSSLOpenSSL.pas: added the TLS version tls1_3. Removed some unused functions while loading the openssl libraries.
IdSSLOpenSSLHeaders & IdSSLOpenSSLHeaders_static: added new functions for openssl 3, remove some unused functions and rename others.

Update the OpenSSL IOHandler to support the OpenSSL 3 libraries. Changes done:

IdSSLOpenSSL.pas: added the TLS version tls1_3. Removed some unused functions while loading the openssl libraries.
IdSSLOpenSSLHeaders & IdSSLOpenSSLHeaders_static: added new functions for openssl 3, remove some unused functions and rename others.
@esegece-com
Copy link
Collaborator Author

Previous pull request was merged by error, I submit a new pull request for openssl 3.

{$NODEFINE SSL_CTX}
SSL_CTX = record
libctx: Pointer;
method: PSSL_METHOD;
cipher_list: PSTACK_OF_SSL_CIPHER;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We aren't meant to access the SSL_CTX structure fields directly. You can use the SSL_CTX_get_ex_data and SSL_CTX_set_ex_data to pass callback data for the IndySSL_CTX_use_PrivateKey_file_PKCS12, IndySSL_CTX_use_certificate_file_PKCS12, IndySSL_CTX_use_PrivateKey_file, IndySSL_CTX_use_certificate_file, and IndySSL_CTX_use_certificate_chain_file.

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