-
Notifications
You must be signed in to change notification settings - Fork 851
Http2 to origin 10 dev #8448
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
Http2 to origin 10 dev #8448
Conversation
|
The AuTest fails because 10-Dev needs to be rebased to contain the fix for the newer jsonschema handling: I'll work on rebasing 10-Dev. |
In the HttpSM, _netvc is set when a connection is made. If the connection fails, however, the _netvc may not be set. This patch handles this situation rather than tripping an assertion if _netvc is nullptr when handling the NET_EVENT_OPEN_FAILED event.
98152a6 to
01d7063
Compare
|
We have an HPACK corruption issue when we ignore response headers on H2 to origin, this was observed on a Pulse Secure VTM.
Because we ignored the response headers for request A in 5, we missed updating the HPACK table. I think we're safe on the client to proxy side, as it seems that we're processing all request headers in sequence and never drop before HPACK processing. |
|
This pull request has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community. |
A continuation of PR #7622 but against the 10-Dev branch instead of master. Includes fixes from several months of production testing.