Skip to content

Fix HTTP/2 ALPN e2e test#90

Merged
h4x3rotab merged 1 commit intomainfrom
vk/d9b5-dstack-ingress-2
Mar 27, 2026
Merged

Fix HTTP/2 ALPN e2e test#90
h4x3rotab merged 1 commit intomainfrom
vk/d9b5-dstack-ingress-2

Conversation

@h4x3rotab
Copy link
Copy Markdown
Contributor

Summary

  • Fix false-negative in the HTTP/2 ALPN e2e test by using openssl s_client -alpn h2 instead of curl --http2 -w '%{http_version}'
  • curl reports version 0 when grpcbin doesn't serve a proper HTTP response on GET /, causing the test to fail even though ALPN negotiation works correctly
  • Verified with full e2e run: 16/16 tests pass

Test plan

  • Full e2e test on Phala CVM — 16 passed, 0 failed

🤖 Generated with Claude Code

curl --http2 reports version 0 when the backend (grpcbin) doesn't
serve a proper HTTP response on GET /. Switch to openssl s_client
-alpn h2 which tests TLS-level ALPN negotiation directly — the
thing we actually care about.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@h4x3rotab h4x3rotab merged commit 781869b into main Mar 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant