Skip to content
This repository was archived by the owner on Oct 21, 2025. It is now read-only.

Add price validation to UiPoolDataProviderV3#196

Open
sqhell wants to merge 1 commit intoaave:masterfrom
sqhell:fix/ui-pool-data-provider-validation
Open

Add price validation to UiPoolDataProviderV3#196
sqhell wants to merge 1 commit intoaave:masterfrom
sqhell:fix/ui-pool-data-provider-validation

Conversation

@sqhell
Copy link

@sqhell sqhell commented Jul 30, 2025

  • Implement MAX_REASONABLE_PRICE constant (1e15) for bounds checking
  • Add validation after oracle price fetches to prevent extreme values
  • Handle zero/invalid oracle addresses gracefully
  • Return zero price for extreme values instead of propagating errors

Fixes GitHub Issue #989 where UiPoolDataProvider returned extreme prices like 1.539e+35 on Arbitrum Sepolia due to oracle configuration issues.

With this validation, prices are bounded to reasonable ranges while the underlying oracle issues are resolved in the deployment configs.

Part of coordinated fix across aave-v3-deploy, aave-address-book, and aave-v3-periphery repositories.

Related PRs:
#196
aave/aave-v3-deploy#77

- Implement MAX_REASONABLE_PRICE constant (1e15) for bounds checking
- Add validation after oracle price fetches to prevent extreme values
- Handle zero/invalid oracle addresses gracefully
- Return zero price for extreme values instead of propagating errors

Fixes GitHub Issue #989 where UiPoolDataProvider returned extreme
prices like 1.539e+35 on Arbitrum Sepolia due to oracle configuration issues.

With this validation, prices are bounded to reasonable ranges while
the underlying oracle issues are resolved in the deployment configs.

Part of coordinated fix across aave-v3-deploy, aave-address-book,
and aave-v3-periphery repositories.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant