fix: throw an error when allow list address is the zero address

yearn / yearn-sdk

🦧 SDK for the yearn.finance platform. WIP

https://npm.im/@yfi/sdk

MIT License

53 stars 56 forks source link

fix: throw an error when allow list address is the zero address #259

Closed jstashh closed 2 years ago

jstashh commented 2 years ago

Description

The address being fetched from the address provider for the AllowList service on mainnet is missing from the address provider. However there is no check for the zero address, so the validation always fails

Related Issue

When trying to deposit/withdraw on mainnet the following error happens:

telegram-cloud-photo-size-4-5895619775663552672-x

Motivation and Context

To be able to deposit/withdraw

How Has This Been Tested?

Running it with a local version of the website

github-actions[bot] commented 2 years ago

size-limit report 📦

Path	Size
dist/sdk.cjs.production.min.js	43.48 KB (+0.04% 🔺)
dist/sdk.esm.js	43.75 KB (+0.03% 🔺)