decentraland / eth-connect

68%

master: 68%

LAST BUILD ON BRANCH 6.3.1

branch: 6.3.1

CHANGE BRANCH

x

Reset

• 6.3.1
• 6.4.0
• agusaldasoro-patch-1
• bignumber
• bump-typescript
• chore/add-more-tests
• chore/align-oidc-node24
• chore/enable-cors
• chore/update-node-to-24
• dependabot/add-v2-config-file
• dependabot/npm_and_yarn/ansi-regex-and-ansi-regex-5.0.1
• dependabot/npm_and_yarn/coveralls-3.1.1
• dependabot/npm_and_yarn/elliptic-6.5.4
• dependabot/npm_and_yarn/istanbuljs/nyc-config-typescript-1.0.2
• dependabot/npm_and_yarn/json5-2.2.3
• dependabot/npm_and_yarn/microsoft/api-documenter-7.12.11
• dependabot/npm_and_yarn/microsoft/api-documenter-7.12.12
• dependabot/npm_and_yarn/microsoft/api-documenter-7.13.34
• dependabot/npm_and_yarn/microsoft/api-documenter-7.13.7
• dependabot/npm_and_yarn/microsoft/api-documenter-7.22.19
• dependabot/npm_and_yarn/microsoft/api-extractor-7.13.2
• dependabot/npm_and_yarn/microsoft/api-extractor-7.13.3
• dependabot/npm_and_yarn/microsoft/api-extractor-7.13.4
• dependabot/npm_and_yarn/microsoft/api-extractor-7.13.5
• dependabot/npm_and_yarn/microsoft/api-extractor-7.14.0
• dependabot/npm_and_yarn/microsoft/api-extractor-7.15.0
• dependabot/npm_and_yarn/mocha-8.3.1
• dependabot/npm_and_yarn/mocha-8.3.2
• dependabot/npm_and_yarn/nanoid-and-mocha-3.3.3
• dependabot/npm_and_yarn/qs-6.5.3
• dependabot/npm_and_yarn/rollup-2.39.1
• dependabot/npm_and_yarn/rollup-2.40.0
• dependabot/npm_and_yarn/rollup-2.41.0
• dependabot/npm_and_yarn/rollup-2.41.1
• dependabot/npm_and_yarn/rollup-2.41.2
• dependabot/npm_and_yarn/rollup-2.41.3
• dependabot/npm_and_yarn/rollup-2.41.4
• dependabot/npm_and_yarn/rollup-2.41.5
• dependabot/npm_and_yarn/rollup-2.42.0
• dependabot/npm_and_yarn/rollup-2.42.2
• dependabot/npm_and_yarn/rollup-2.42.3
• dependabot/npm_and_yarn/rollup-2.42.4
• dependabot/npm_and_yarn/rollup-2.46.0
• dependabot/npm_and_yarn/rollup-2.50.6
• dependabot/npm_and_yarn/rollup-2.51.2
• dependabot/npm_and_yarn/rollup-2.56.0
• dependabot/npm_and_yarn/ts-node-10.1.0
• dependabot/npm_and_yarn/ts-node-10.9.1
• dependabot/npm_and_yarn/types/mocha-10.0.1
• dependabot/npm_and_yarn/types/mocha-8.2.2
• dependabot/npm_and_yarn/types/node-fetch-2.6.4
• dependabot/npm_and_yarn/typescript-4.2.2
• dependabot/npm_and_yarn/typescript-4.2.3
• dependabot/npm_and_yarn/typescript-4.2.4
• dependabot/npm_and_yarn/typescript-4.3.5
• dependabot/npm_and_yarn/websocket-1.0.34
• dependabot/npm_and_yarn/y18n-4.0.1
• expect
• feat/add-solidity-function
• feat/add-to-payload-to-contract-fn
• feat/build-individual-files
• feat/fetchfunction-accept-native-fetch
• feat/sig
• feat/support-tupples
• feat/support-tupples-mz
• feat/update-geth-make-file
• fix/byte32-function-params
• fix/httpprovider-cancel-error-response-body
• fix/personal_sign
• fixes
• lint
• master
• nearnshaw-patch-1
• pr
• provenance
• refs/tags/5.0.1
• refs/tags/5.1.0
• refs/tags/5.1.1
• refs/tags/6.0.0
• refs/tags/6.0.1
• refs/tags/6.0.2
• refs/tags/6.0.3
• refs/tags/6.1.0
• refs/tags/6.2.0
• refs/tags/6.2.1
• refs/tags/6.2.2
• refs/tags/6.2.4
• security/pin-github-actions-sha
• test-dg
• test/utf8
• test2
• update-tests
• updates
• workflows

Build # 28115754140

Build Type

push

github

Committed by web-flow

Commit Message

fix: release the response body on non-2xx HTTPProvider responses (#582)

HTTPProvider.sendAsync discarded the fetch Response on a non-2xx status without
reading its body. With native fetch (undici) and browser fetch the underlying
socket stays checked out until the body is consumed or cancelled, so error
responses (e.g. an RPC endpoint returning 429/5xx) leaked connections under load.
node-fetch's default non-keep-alive agent masked this previously.

Drain the body via text() before invoking the callback (releasing the connection
across all fetch implementations) and surface the server's error detail in the
thrown error, bounded to 512 chars and best-effort, so non-2xx failures carry
useful diagnostics instead of a bare status code. The 2xx path is unchanged (it
already consumes the body via json()).

Adds unit tests covering the drain, error-detail surfacing, cleanup-failure, and
success paths.

Coverage Stats

568 of 995 branches covered (57.09%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 1 file covered. (100.0%)

1293 of 1754 relevant lines covered (73.72%)

148.33 hits per line