elixir-mint / mint

88%

main: 88%

LAST BUILD ON BRANCH ericmj/local-h2c-integration-tests

branch: ericmj/local-h2c-integration-tests

CHANGE BRANCH

x

Reset

Sync Branches

• ericmj/local-h2c-integration-tests
• al/1.18
• al/ci
• al/client-settings
• al/deps
• al/elixir-1.10
• al/elixir-1.11+
• al/fix-leak
• al/fix-tests
• al/http2-logs
• al/improve-defguard
• al/modernize-ci
• al/stream-docs
• al/update-ci-versions
• al/updates
• allow-not-validating-target
• allow_302_for_twitter_integration_test
• andrea/code-coverage
• andrea/dialyzer
• andrea/fix-guard
• andrea/new-ci-versions
• comp-time-optimization
• configurable-test-ports
• connection-docs-ref
• contributing-and-testing-housekeeping
• docs-for-open
• dont-close-socket-on-recv-timeout
• ericmj/deprecate-read-write
• ericmj/fix-1xx-informational-response
• ericmj/http2-connection-window-size
• ericmj/http2-larger-default-windows
• ericmj/inet4-option
• ericmj/support-elixir-1.12
• ericmj/update-ubuntu
• fix-cannot-build-docker-image-using-podman-compose
• fix-dialyzer
• fix-http1-tests-not-tagged-correctly
• fix-incorrect-ci-badge
• fix-warnings
• fix/humize_error_code_custom_error
• http1-reason-phrase
• ip-address-match-fun
• jv-default-store
• jv-elixir-1-19
• main
• next_body_chunk
• np/export-error-types
• optimize-http/2-request-creation
• patch-3
• preserve_header_case
• pvthuyen/add-support-for-cacerts-get
• refs/heads/main
• refs/pull/399/merge
• refs/pull/407/merge
• rewrite-contributing-and-testing-doc
• run-ci-against-erlang-27.1
• unnecessary-rst-stream-2
• update-deps
• use_local_http_bin_in_all_tests
• whatyouhide/issue311
• wm-decompression
• wm-docker-compose
• wm-recv-response
• wrap-tunnel-errors

Build # ef5fd4b6b8872516891d56b0dc8b9954b8b996e6-PR-482

Build Type

Pull #482

github

Committed by ericmj

Commit Message

Fix stray :ssl message leak in HTTP/2 integration tests

The twitter.com describe block was using @moduletag instead of
@describetag, which made its connect: {"twitter.com", 443} tag apply
to the whole module. That caused the top-level setup to open an HTTPS
connection to twitter.com even for tests outside the describe (like
the TCP h2c test), and the SSL socket's SETTINGS frame would land in
the test mailbox. The previous test had a swallow-hack comment about
"SSL messages sneaking in"; this removes the root cause.

Pull Request Pull Request #482: Run HTTP/2 integration tests against local Caddy instead of nghttp2.org

Coverage Stats

1290 of 1472 relevant lines covered (87.64%)

248.64 hits per line