New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Default authorisation when adding LP is too high and unchangeable #7839

Open

cnukaus opened this issue Feb 8, 2025 · 1 comment

cnukaus commented Feb 8, 2025

When adding LP to a new pool (v4 and v3), the default authorisation is basically uncapped... and users can't change the amount in signature.

This is not safe.

Author

cnukaus commented Feb 8, 2025

MetaMask Version 12.6.2

Tested in V4,

Looks like useV3DerivedMintInfo (or a similar function) is unable to read currencyBalances

interface/apps/web/src/pages/AddLiquidityV3/index.tsx

Line 74 in 6e27a64

useV3DerivedMintInfo,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment