-
Notifications
You must be signed in to change notification settings - Fork 778
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
Event handles with VC_EVENT_ERROR but not VC_EVENT_EOS #11358
Comments
I'm not sure whether QUICNextProtocolAccept.cc receives EOS, but we can add it. Is the list complete? I wonder why SSLNextProtocolAccept.cc isn't on the list. |
Its likely not complete, just me staring at the code |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
While reviewing #11346, I went through and found other places where the EOS event was not handled but ERROR was:
I wonder, should we add EOS to these switch statements as well?
The text was updated successfully, but these errors were encountered: