-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fix OpenSSL error handling for EOF (support for OpenSSL 3.2) #14169
Fix OpenSSL error handling for EOF (support for OpenSSL 3.2) #14169
Conversation
On Windows it looks like you have to check |
Co-authored-by: Beta Ziliani <beta@manas.tech>
Can't we add OpenSSL 3.2 to the OpenSSL CI to check this change? |
We could, but it requires some non-trivial additions to the CI configuration. |
…rystal-lang#14169)" This reverts commit 7df0b2e.
This got eventually reverted in #14217 due to #14200 (comment) A new approach is happening in #14219 |
This is following the reasoning from #14168 (comment)
I'm not entirely sure this is exactly correct API usage. The documentation is not very extensive. But judging from the resulting behaviour, and backed by available documentation it seems to be more correct than the current implementation.