elixir-mint / mint

88%

main: 88%

LAST BUILD ON BRANCH optimize-http/2-request-creation

branch: optimize-http/2-request-creation

CHANGE BRANCH

x

Reset

Sync Branches

• optimize-http/2-request-creation
• 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/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
• connection-docs-ref
• contributing-and-testing-housekeeping
• docs-for-open
• ericmj/deprecate-read-write
• ericmj/inet4-option
• ericmj/update-ubuntu
• fix-cannot-build-docker-image-using-podman-compose
• fix-http1-tests-not-tagged-correctly
• fix-incorrect-ci-badge
• fix/humize_error_code_custom_error
• ip-address-match-fun
• jv-default-store
• jv-elixir-1-19
• main
• 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 # f8ece36599321fd1aa0e8681dd785b7b423037e5-PR-423

Build Type

Pull #423

github

Committed by the-mikedavis

Commit Message

HTTP/2: Optimize check for CONNECT method in header creation

`String.upcase/1` by default iterates over the graphemes in the
input string which can be relatively slow. Since we're only interested
in case-insensitive equality, we can write a small custom equality
checking function that can avoid creating new terms and can return
early.

Using a charlist is slightly faster than passing `"CONNECT"` as a
binary.

Pull Request Pull Request #423: Optimize HTTP/2 request creation

Run Details

10 of 11 new or added lines in 1 file covered. (90.91%)

1254 of 1432 relevant lines covered (87.57%)

263.18 hits per line