Open akotulu opened 3 months ago
Doesn't this change break any user code which uses the SSLv23 ones?
Here is quote from docs.
SSLv23_method(), SSLv23_server_method(), SSLv23_client_method() These functions do not exist anymore, they have been renamed to TLS_method(), TLS_server_method() and TLS_client_method() respectively. Currently, the old function calls are renamed to the corresponding new ones by preprocessor macros, to ensure that existing code which uses the old function names still compiles. However, using the old function names is deprecated and new code should call the new functions instead.
Ok, but you do change the enum provided by ACE in such a way that when someone uses it, it will not compile unless they change their code
Ok, I will add the enum values back with corresponding TLS calls.
Mark the old enums as deprecated so that we know they are deprecated and can be removed at some point, maybe use the C++14 https://en.cppreference.com/w/cpp/language/attributes/deprecated, C++17 is now required for ACE/TAO
Made the changes. Is it ok now?
What when ssl version is smaller as 0x10100000L, than the new defines are there. Also in ACE_SSL_Context::load_trusted_ca
the old enums are not tested
Huh, sorry about the mess. Haven't done such a backwards compatibility stuff before. It should now be ok.
When < 0x10100000L the enums will still give a deprecated warning
Idk if this is the best way to remove the deprecation warning, but here it is.
Please fix fuzz errors
Done.
SSLv23_method(), SSLv23_server_method() and SSLv23_client_method() were deprecated and the preferred TLS_method(), TLS_server_method() and TLS_client_method() functions were added in OpenSSL 1.1.0.
All version-specific methods were deprecated in OpenSSL 1.1.0.
SSLv23_method