-
-
Notifications
You must be signed in to change notification settings - Fork 9.9k
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
QUIC APL: Tick on SSL_read failure in non-blocking mode #21381
Conversation
@hlandau Do we want a testcase that exercises this? |
I spotted this problem while writing the test in #21367. There's a code comment about it in that test along with a workaround. I'd suggest we merge this as-is, and then I can update #21367 to remove the work around and that should serve as a test. Changing this to "tests:deferred" |
24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually. |
Merged. |
... Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from #21381)
As discussed, SSL_read in non-blocking mode should tick if we have no data to return.
Fixes #21365.