Skip to content
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.

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

Transactions submitted with 0Gwei gas price | Sokol #52

Closed
RyRy79261 opened this issue Aug 9, 2019 · 9 comments
Closed

Transactions submitted with 0Gwei gas price | Sokol #52

RyRy79261 opened this issue Aug 9, 2019 · 9 comments

Comments

@RyRy79261
Copy link

Description

Intermittently on Sokol, transactions will be submitted with a gas price of 0, even with explicit gas price of 1 being set. const gasPrice = anyToBN(0x3B9ACA00);

Forwarding 0.02Eth to an account: https://blockscout.com/poa/sokol/tx/0x3cb36122abbea502770acab0d0ad8c1a63a8480322ea599bc130b3880fd407b4/internal_transactions

Account deploy: https://blockscout.com/poa/sokol/tx/0x7c702258a310f4d4ac5a179bfdde19d96af8c73fd6357fce0779e5e541c8487d/internal_transactions

Account deploy: https://blockscout.com/poa/sokol/tx/0xfccc3842a1b7f9b82614154960847f358bed01b18dfb3d0524a890fbfe5f8ff4/internal_transactions

Account deploy: https://blockscout.com/poa/sokol/tx/0x11f0ba2e3fbf8872dbef5d23a1318d274e4a167cc5a8d3c56426a9224f89f5b9/internal_transactions

@RyRy79261
Copy link
Author

I have reproduced it on the Sokol playground.

Steps

  • Init from fresh
  • Create Account
  • Estimate deploy of account
  • Send funds to account address
  • Trigger deploy account
  • A hash is returned
  • Transaction is inspected on blockscout

PVT: 0x36c5d5fc99e74030f15f15171d9902d3b677b1a39d4a5327a2ca550ffb3634f7
Resulting tx from deploy: https://blockscout.com/poa/sokol/tx/0x43585f5d323d9a5b1228df20f41e8c3d0dca400cd15f78d298b289bd615f8a04/internal_transactions

@RyRy79261
Copy link
Author

RyRy79261 commented Aug 11, 2019

I've found that the estimate is being generated with gas estimates of 0 for transactions.

The estimation occurs at the relayer so this would need to be addressed there.

@stanislaw-glogowski
Copy link
Contributor

There is no possibility to submit gas price anymore. In latest version you can choose gasPriceStategy (Avg, Fast = Avg x 2)

@RyRy79261
Copy link
Author

image

@RyRy79261
Copy link
Author

@stanislaw-glogowski Please reopen the issue.

@RyRy79261
Copy link
Author

I have recreated the issue in the playground
image

@RyRy79261
Copy link
Author

All the console logged estimate objects prior to the hash log had gas rates of 0
image (2)

@RyRy79261
Copy link
Author

The pseudocode for the repair is as follows:

if(networkId == 77 || networkId == 100){
   estimate.gasPrice = anyToBN(1000000000);
}

@stanislaw-glogowski Please apply this to the function that generates estimates in the relayer

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

No branches or pull requests

2 participants