FuelLabs/fuels-rs

InsufficientFeeAmount on test accounts.

Opened this issue · 14 comments

Oleksii-Kshenskyi commented

Hi! So when writing a Rust order matcher with the fuels crate, all my test accounts eventually become unusable with this error message:
[15:11:51][ERROR] An error occurred while matching: IO error: Response errors; Validity(InsufficientFeeAmount { expected: 326087, provided: 323226 })

I'm sorry I cannot provide a specific scenario to reproduce, but it would usually go like this:

  1. I create a new wallet with Fuelet Wallet (Chrome)
  2. I take test Ether from the faucet
  3. I start testing the matcher: creating test orders, matching them, then creating more etc.
  4. Eventually at some point, all blockchain operations with this wallet start erroring out with the error above, with no way to fix it
  5. I create a new account, get some test Ether and start testing the matcher: all is good again and the problem disappears.
  6. At some point, the problem starts happening on the new wallet too and everything repeats again.

I unfortunately don't know of an easier or more reliable way to reproduce this. The only way is, I have a wallet and a private key for which this problem already exists. If needed, I can give it to you: it's a test account, so it's of no importance.

P.S. This is using the fuels crate version 0.55.1 on Manjaro Linux, on Beta 5 (related to my comment in #1342, it's the same issue)

MujkicA commented

Hi, thanks for informing us of this issue.

Since you are testing on beta-5, would you be able to share the relevant wallet address and/or transaction id so that we have some more insight into what is happening here?

Oleksii-Kshenskyi commented

Hi, thanks for informing us of this issue.

Since you are testing on beta-5, would you be able to share the relevant wallet address and/or transaction id so that we have some more insight into what is happening here?

Hi, thanks for the response! Unfortunately, I can't provide the transaction ID, because the transaction's Result returns Error, and the tx_id is in the Ok type (FuelCallResponse)... But I can provide the wallet address, and the "good" part here is that I think almost any transaction on that wallet should return this error, so this should be easy to reproduce.

Here's the wallet address: fuel1aeh0ruymh5j9syvldjm7zxsf2v2tua7pm2fn9hd9vn9ps5gw9txs9m4ngc.
Let me know if I can provide anything else, and thanks for the help!

chlenc commented 
git clone git@github.com:compolabs/spark-matcher-rs.git   
cd spark-matcher-rs
echo «env data» >> .env    
cargo run

.env file

NODE_ENV=development
PORT=5003

PRIVATE_KEY="0x7e07fb62e6db3bddb766c6ef21fc622f1ff93d8b4d7ee7b7e1c8c25a15dbadaf"

CONTRACT_ID="0x0f0c1065a7b82d026069c5cf070b21ee65713fd1ac92ec1d25eacc3100187f78"

INDEXER_URL="https://indexer.spark-defi.com"
FETCH_ORDER_LIMIT=100

MARKET="BTC"

LOG_FILE="matcher.log"
FILE_LOG_LEVEL="debug"
CONSOLE_LOG_LEVEL="info"

MAX_FAIL_COUNT=3
Oleksii-Kshenskyi commented

@MujkicA @hal3e Guys, I've tried the suggested workaround for estimating transaction cost and bumping it by ~20%, but unfortunately the InsufficientFeeAmount pops up even for the estimate_transaction_cost() itself. So I need the estimation, but the estimation itself triggers the issue. What can I do in this case?
bork

Here, I get the Err while unwrapping on line 257.

  • 👀1
MujkicA commented

You can try using the wallets entire base asset balance as temporary workaround

let wallet = Wallet::from_address(addr, Some(provider.clone()));
let balance = provider.get_asset_balance(wallet.address(), BASE_ASSET_ID).await?;
call_handler.add_custom_asset(BASE_ASSET_ID, balance, None);
Oleksii-Kshenskyi commented

We (me and @MujkicA) have found a solid workaround for this issue, so thanks so much for all the help! Won't be closing the issue until the official fix for this is out. Thank you guys.

Nazeeh21 commented

Hey guys, how does the progress on this look like?

JasoonS commented

I'm also running into this issue, I'm unable to interact with any contracts I deploy on testnet via rust.

Here is my code for a local reproduction (it is very simple): https://github.com/enviodev/fuel-greeter/pull/1/files#r1616211590

MujkicA commented

I'm also running into this issue, I'm unable to interact with any contracts I deploy on testnet via rust.

Here is my code for a local reproduction (it is very simple): https://github.com/enviodev/fuel-greeter/pull/1/files#r1616211590

The workaround for now is to manually set a high script gas limit on your contract call via the TxPolicies.

Here is a rough example:

let gas_limit = provider.consensus_parameters().max_gas_per_tx() * 0.8;
let res = instance
              .methods()
              .your_method().with_tx_policies(TxPolicies::default().with_script_gas_limit(gas_limit)))
              .call()
              .await?;
  • 👍3
JasoonS commented

Thank you this is working for me.

max_gas_per_tx wasn't found for me, but just hard-coding gas_limit to 300000 works well.

Appreciate the help 🙏

  • 👍1
mpoplavkov commented

I have a similar error but with much higher gas limits :

Validity(InsufficientFeeAmount { expected: 1000000000000, provided: 1000000000 })

The function is trivial:

    fn initialize() {
        let sender = msg_sender().unwrap();
        initialize_ownership(sender);
    }

When I set script gas limit as suggested above to the expected value 1000000000000, I'm getting another error:

Transaction(Validation("TransactionMaxGasExceeded"))
MujkicA commented

When I set script gas limit as suggested above to the expected value 1000000000000, I'm getting another error:

Does the error persist if you set the script gas limit to:

let gas_limit = provider.consensus_parameters().tx_params().max_gas_per_tx() * 0.8;
mpoplavkov commented

Does the error persist if you set the script gas limit to:

let gas_limit = provider.consensus_parameters().tx_params().max_gas_per_tx() * 0.8;

The issue is solved for me after upgrading the SDK to 0.63.0

  • 👍1