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

Revert eth: Ensure maxFeePerGas in all transactions never exceed -masGasPrice #2163

Merged
merged 2 commits into from
Jan 5, 2022

Conversation

leszko
Copy link
Contributor

@leszko leszko commented Jan 5, 2022

The following PR: #2111 caused the following bug: #2152

We revert it for now and will implement maxGasPrice in a different manner.

What does this pull request do? Explain your changes. (required)
N/A

Specific updates (required)
N/A

How did you test each of these updates (required)
Tested that it fixes the related bug.

Does this pull request close any open issues?
fix #2152

Checklist:

…sPrice defined by the user"

This reverts commit d7488f2.
@leszko leszko requested a review from yondonfu January 5, 2022 15:53
Copy link
Member

@yondonfu yondonfu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yondonfu yondonfu merged commit 34185c0 into livepeer:master Jan 5, 2022
@0xVires
Copy link

0xVires commented Jan 5, 2022

just a comment here: If it's too much work to find the issue, I'd say don't bother considering the L2 migration is only 1-2 months out. Gas price won't be an issue there anyways so not sure if it's worth it spending too much time on this :)

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

Successfully merging this pull request may close these issues.

Session crashes (without errors in log) right after the O sends a tx
3 participants