Skip to content

ci: use self-hosted httpbun in tests#2568

Merged
AlexV525 merged 2 commits into
mainfrom
ci/httpbun-self-hosted-tests
Jul 18, 2026
Merged

ci: use self-hosted httpbun in tests#2568
AlexV525 merged 2 commits into
mainfrom
ci/httpbun-self-hosted-tests

Conversation

@AlexV525

@AlexV525 AlexV525 commented Jul 18, 2026

Copy link
Copy Markdown
Member

Motivation

httpbun.com is currently returning HTTP 502 responses, which makes the integration tests fail independently of dio changes. The failure is visible in the PR #2567 stable job. CI already starts httpbun.local, but HTTP/2 pinning and all browser tests still used the public service.

upstream sharat87/httpbun#35

Changes

  • Reuse the shared test-server URL in HTTP/2 pinning tests and generate their certificate fingerprint from the configured host.
  • Keep local HTTPS for VM and HTTP/2 tests.
  • Start a second local HTTP container for Chrome, Firefox, and Flutter tests, avoiding browser-specific temporary-CA trust configuration.
  • Keep httpbun.com as the default outside the CI-local setup.
  • Avoid assuming onBadCertificate and validateCertificate receive different certificates; that varies with the certificate chain and trust store.

This touches TLS and certificate-pinning test infrastructure only; production TLS behavior is unchanged. Maintainers: please apply the extra scrutiny required for this sensitive area.

Verification

melos run format and melos run analyze pass. The local Chrome shared suite passes 54 tests with one existing skip against the HTTP container, including CORS, redirects, status codes, timeouts, and uploads. GitHub Actions passes on the minimum, stable, and beta SDK jobs, covering VM, Chrome, Firefox, and Flutter tests; the stable job also passes APK build and coverage generation.

Implementation, tests, and local review by Codex.

AlexV525 and others added 2 commits July 18, 2026 17:15
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
@AlexV525 AlexV525 changed the title ci: use self-hosted httpbun for pinning tests ci: use self-hosted httpbun in tests Jul 18, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Code Coverage Report: Only Changed Files listed

Package Base Coverage New Coverage Difference
plugins/web_adapter/lib/src/cors.dart 🟢 100% 🟢 96.55% 🔴 -3.45%
Overall Coverage 🟢 85.73% 🟢 85.68% 🔴 -0.05%

Minimum allowed coverage is 0%, this run produced 85.68%

@AlexV525
AlexV525 marked this pull request as ready for review July 18, 2026 09:39
@AlexV525
AlexV525 requested a review from a team as a code owner July 18, 2026 09:39
@AlexV525
AlexV525 merged commit c655c5e into main Jul 18, 2026
5 checks passed
@AlexV525
AlexV525 deleted the ci/httpbun-self-hosted-tests branch July 18, 2026 09:59
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