Skip to content

Extended's 1 MB market list was cut off by a 15 s client timeout - #3036

Merged
Flotapponnier merged 1 commit into
devfrom
fix/extended-timeout
Oct 18, 2026
Merged

Flotapponnier merged 1 commit into
devfrom
fix/extended-timeout

Conversation

@Flotapponnier

Copy link
Copy Markdown
Collaborator

api.starknet.extended.exchange/api/v1/info/markets returns ~1 MB and takes 4 to 15 s from the VPS (three curls: 6.1 s, 4.4 s, 14.6 s). The client's 15 s timeout expired mid-body, io.ReadAll's error was discarded, and the truncated body surfaced as err parse: unexpected end of JSON input. Extended's perp_venue_health dipped to 0.5 on those ticks (min over 24 h: 0.5).

  • timeout 15 s → 45 s (sources run concurrently in a 60 s tick)
  • a failed body read is now reported as read_error, not a parse error

Found while monitoring the #3035 deploy. go build, go vet, go test green.

/info/markets returns about 1 MB and takes 4 to 15 s from the VPS. The
client's 15 s budget expired mid-body, io.ReadAll's error was dropped,
and the truncated JSON was logged as a parse error: perp_venue_health for
Extended fell to 0.5 on those ticks. The budget is 45 s (sources run
concurrently inside a 60 s tick) and a short read is reported as one.
@Flotapponnier
Flotapponnier merged commit f59ce7f into dev Oct 18, 2026
2 checks passed
Flotapponnier added a commit that referenced this pull request Oct 18, 2026
release: Gains and Lighter back on the perp boards (cherry-pick #2963, #3035, #3036)
bakabhai993 pushed a commit to bakabhai993/OpenChainBench that referenced this pull request Oct 18, 2026
…inBench#3036)

/info/markets returns about 1 MB and takes 4 to 15 s from the VPS. The
client's 15 s budget expired mid-body, io.ReadAll's error was dropped,
and the truncated JSON was logged as a parse error: perp_venue_health for
Extended fell to 0.5 on those ticks. The budget is 45 s (sources run
concurrently inside a 60 s tick) and a short read is reported as one.

(cherry picked from commit f59ce7f)
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