-
-
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: Raise more errors when appropriate #21700
Conversation
CI failures are relevant |
Fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Generally good stuff. @mattcaswell's comments apply. The TXP change needs to be reverted and all raising in quic_impl.c
should use one of the RAISE macros, defining a new non-I/O one if desired.
This was already resolved by openssl#21547
This improves tracking where the failure was triggered.
Raise errors when appropriate.
Even in case of later failure we need to flush the previous packets.
Ready for re-review. |
Ping @openssl/committers for second review |
This pull request is ready to merge |
Merged to master branch. Thank you for the reviews. |
This improves tracking where the failure was triggered. Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from #21700)
Raise errors when appropriate. Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from #21700)
Even in case of later failure we need to flush the previous packets. Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from #21700)
Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> (Merged from #21700)
There are probably more places where we could raise errors but this should be sufficient for MVP.