Added Autest for H2 CONNECT and fix a crash triggered by the test #9781
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Note: The updated |
Co-authored-by: Masakazu Kitajo <maskit@apache.org>
lzx404243
force-pushed
the
h2-connect-pv-fix
branch
from
218f0a3 to
745c61a
Compare
Just to record it, Proxy Verifier was updated to the required v2.8.1 via: #9834 |
moonchen
approved these changes
Jun 23, 2023
JosiahWI
pushed a commit
to JosiahWI/trafficserver
that referenced
this pull request
Jul 19, 2023
These are cherry-picks from asf that has the fix for crash caused by H2 CONNECT. * Added http connect Autest with proxy verifier (apache#9315) * Added a HTTP CONNECT test using proxy verifier * Updated comment * added proxy-response verification (cherry picked from commit b5f2023) * Added Autest for H2 CONNECT and fix a crash triggered by the test (apache#9781) In apache#9616, @maskit wrote an H2 CONNECT Autest but couldn't include that in the final PR because of a Proxy Verifier issue. Now that the Proxy Verifier issue is resolved, the Autest is added in this PR(with a few tweaks). ATS crashes with the new test executing HTTP/2 tunneling traffic. This PR also includes a fix to resolve that. (cherry picked from commit df7ccfe) * Update to Proxy Verifier v2.8.1 (apache#9834) Proxy Verifier v2.8.1 has fixes for the way Proxy Verifier relates to HTTP/2 CONNECT method request pseudo header fields. This will be helpful for testing HTTP/2 CONNECT requests. (cherry picked from commit d5c47a7)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In #9616, @maskit wrote an H2 CONNECT Autest but couldn't include that in the final PR because of a Proxy Verifier issue.
Now that the Proxy Verifier issue is resolved, the Autest is added in this PR(with a few tweaks).
ATS crashes with the new test executing HTTP/2 tunneling traffic. This PR also includes a fix to resolve that.