Skip to content

fix(stables): retry DeFiLlama price requests - #372

Merged
spalen0 merged 1 commit into
mainfrom
llama-price
Sep 22, 2026
Merged

spalen0 merged 1 commit into
mainfrom
llama-price

Conversation

@spalen0

@spalen0 spalen0 commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • route DeFiLlama current-price requests through the shared retrying HTTP client
  • retry transient TLS/connection errors, timeouts, HTTP 429s, and 5xx responses
  • remove the now-unused defillama-sdk dependency
  • add regression coverage for recovery from an SSLError and price response parsing

Persistent failures still propagate after the retry budget is exhausted, preserving the existing crash alert.

Validation

  • uv run --extra dev pytest tests/ -q — 986 passed, 6 skipped, 5 subtests passed
  • uv run --extra dev ruff check utils/defillama.py tests/test_http_client.py tests/test_utils.py
  • uv lock --check
  • git diff --check

Fixes #371

spalen0

This comment was marked as off-topic.

@spalen0
spalen0 merged commit b47d321 into main Sep 22, 2026
3 checks passed
@spalen0
spalen0 deleted the llama-price branch September 22, 2026 13:55
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.

[pegs] stables-oracles crashes on transient TLS EOF (no retry in utils/defillama.py)

1 participant