Transaction underpriced #1703
Comments
that's rather an issue with gasprice ref ethers-io/ethers.js#2828 perhaps that's an issue with the rpc so an insufficient gasprice was set. deploying on Mumbai works? |
|
Same issue with Polygon Mainnet (mumbai is OK), guess it's related issue with ethers-io/ethers.js#2828 (comment). I am using |
|
@mattsse I haven't found a more recent issue on this. The problem may be the minimum 30 gwei priority fee on Polygon Mainnet (as described in ethers-io/ethers.js#3370). Is this something that can be patched in Foundry? |
|
interesting, definitely, we can enforce this. since this is mandatory. I think we should set it by default if not provided |
|
running into the same issue, can confirm that adding --legacy tack, ie:
is an acceptable current workaround for anyone also encountering this issue |
|
@robriks, is it happening again? Have you updated Foundry? |
|
updating foundry fixed this. thanks for checking in! |
|
this has since been fixed with dedicated polygon gas station |
|
|
|
Using Mumbai, I'm still intermittently having issues with gas estimation. I can usually get transactions posted using --legacy or --skip-simulation and --gas-estimate-multiplier |
### Description After investigating #2959, I found the following this is a known problem with Polygon, explanation here ethers-io/ethers.js#2828 (comment) Fun fact Asa looked into this once #771 Here's a discussion it in Foundry, which I found hoping that ethers-rs folks had ran into this before foundry-rs/foundry#1703 Foundry fixed this by using the Polygon gas station oracle which seems to be recommended path https://github.com/foundry-rs/foundry/pull/3368/files#diff-c89a4bbf7a90da118dcf00c5fe70eba78f8e5d95662bb5f039a353113e95042bR205 There's actually a polygon ethers middleware for this https://docs.rs/ethers/latest/ethers/middleware/gas_oracle/polygon/struct.Polygon.html So I (originally) borrowed this code from Foundry https://github.com/foundry-rs/foundry/blob/master/crates/common/src/provider.rs#L254-L290 Changed to use Middlewares This also means we can remove our existing janky Polygon logic ### Drive-by changes <!-- Are there any minor or drive-by changes also included? --> ### Related issues <!-- - Fixes #[issue number here] --> ### Backward compatibility <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> ### Testing <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->
After investigating #2959, I found the following this is a known problem with Polygon, explanation here ethers-io/ethers.js#2828 (comment) Fun fact Asa looked into this once #771 Here's a discussion it in Foundry, which I found hoping that ethers-rs folks had ran into this before foundry-rs/foundry#1703 Foundry fixed this by using the Polygon gas station oracle which seems to be recommended path https://github.com/foundry-rs/foundry/pull/3368/files#diff-c89a4bbf7a90da118dcf00c5fe70eba78f8e5d95662bb5f039a353113e95042bR205 There's actually a polygon ethers middleware for this https://docs.rs/ethers/latest/ethers/middleware/gas_oracle/polygon/struct.Polygon.html So I (originally) borrowed this code from Foundry https://github.com/foundry-rs/foundry/blob/master/crates/common/src/provider.rs#L254-L290 Changed to use Middlewares This also means we can remove our existing janky Polygon logic <!-- Are there any minor or drive-by changes also included? --> <!-- - Fixes #[issue number here] --> <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->
After investigating #2959, I found the following this is a known problem with Polygon, explanation here ethers-io/ethers.js#2828 (comment) Fun fact Asa looked into this once #771 Here's a discussion it in Foundry, which I found hoping that ethers-rs folks had ran into this before foundry-rs/foundry#1703 Foundry fixed this by using the Polygon gas station oracle which seems to be recommended path https://github.com/foundry-rs/foundry/pull/3368/files#diff-c89a4bbf7a90da118dcf00c5fe70eba78f8e5d95662bb5f039a353113e95042bR205 There's actually a polygon ethers middleware for this https://docs.rs/ethers/latest/ethers/middleware/gas_oracle/polygon/struct.Polygon.html So I (originally) borrowed this code from Foundry https://github.com/foundry-rs/foundry/blob/master/crates/common/src/provider.rs#L254-L290 Changed to use Middlewares This also means we can remove our existing janky Polygon logic <!-- Are there any minor or drive-by changes also included? --> <!-- - Fixes #[issue number here] --> <!-- Are these changes backward compatible? Are there any infrastructure implications, e.g. changes that would prohibit deploying older commits using this infra tooling? Yes/No --> <!-- What kind of testing have these changes undergone? None/Manual/Unit Tests -->
forge create --rpc-url https://polygon-mainnet.g.alchemy.com/v2/NDF-tLzxxxxxxxxxxxx --constructor-args "bushman" "BSH" --private-key af180013952262434794a2cd8xxxxxxb src/NFT.sol:NFT results in "transaction underpriced" error.
Is that a generic term for "insufficient funds in the wallet"? Or is it referring to something else? I was assuming that Polygon would be rather cheap to deploy to.
Is there a way to estimate the deployment cost before creating, if that is the issue?
The text was updated successfully, but these errors were encountered: