alfg / alfg/blockchain

New client has no default timeout and may hang on slow network

Offen
#4 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Go
Sterne
45
Forks
19
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

The New constructor currently returns an http.Client with Timeout=0, which means requests can wait indefinitely on network stalls.

Reproduction command:

sandbox-exec -f /tmp/no-network4.sb env GO111MODULE=off go test -run TestNewClientSetsTimeout -count=1 ./...

Observed output before fix:
- FAIL: expected positive timeout, got 0s

Expected behavior:
- New clients should set a bounded default timeout so requests fail predictably under network issues.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.