Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

Add London support (minus access lists) to Truffle Contract #4214

Merged
merged 1 commit into from
Aug 5, 2021

Conversation

haltman-at
Copy link
Contributor

This PR adds support for London transactions, minus access lists, to Truffle Contract.

This PR is a draft because it needs to be tested. Unfortunately, it can't be tested with Ganache at the moment, because Ganache doesn't support London yet.

Obviously, prefer to wait until Ganache supports London before doing this, but London is coming soon and after discussing things with @gnidan we decided this is important enough not to wait. Access lists are less important, so for now we're still going to wait on that one.

@haltman-at
Copy link
Contributor Author

Hm, so it seems to work, sort of... but, like, it demands that maxFeePerGas and maxPriorityFeePerGas be given as hex strings. Uh, we can work around that if we have to, but in the meantime I'm going to go file an issue with web3...

@haltman-at
Copy link
Contributor Author

Oh, geez, it's worse than that! London transactions don't work at all because web3 adds a default gasPrice field! So web3 1.5.0 is basically just broken... well, I'll go file an issue...

@haltman-at
Copy link
Contributor Author

Filed issues:
web3/web3.js#4210
web3/web3.js#4211

@cds-amal
Copy link
Member

How can I test this with the latest geth @haltman-at ?

@haltman-at
Copy link
Contributor Author

How can I test this with the latest geth @haltman-at ?

Is there a particular thing you're confused about, or are you looking for some sort of walkthrough? In the latter case Slack might be more appropriate here than GitHub comments...

@haltman-at haltman-at marked this pull request as ready for review August 5, 2021 19:42
Copy link
Contributor

@eggplantzzz eggplantzzz left a comment

Choose a reason for hiding this comment

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

I like it!

@haltman-at haltman-at merged commit f1a70cf into develop Aug 5, 2021
@haltman-at haltman-at deleted the truffle-london branch August 5, 2021 21:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants