-
Notifications
You must be signed in to change notification settings - Fork 59
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
Map nonce of transaction when creating contracts #20
Comments
Hello, my name is Dave Mariano. I am fascinated with what Horizon is doing and want to learn more. I'm interested in tackling this issue for you. I have forked your repo and have been poking around. I think the contract creation nonce could be tracked in a different space, as you suggested, by incrementing the current nonce only if the transaction receipt's My plan is to test this and the behaviour of nonces and transaction counts vs number of contracts created locally. Any advice for maintaining a consistent environment is appreciated. Also please see issue #23 as the contribution guidelines are missing. I have a few questions/confirmations to make sure I am understanding the problem correctly. Please forgive my ignorance, I'm eager to contribute and even join the team:
|
Hi @d-mariano, first and foremost thanks for your help, we really appreciate it! 😄
|
@Agusx1211 I appreciate the quick response! Happy Canada Day, by the way! I'm sorry to bother you on a holiday. :) It's good to hear I'm on the right track. I'll try working through this with the LocalRelayer. Thank you for offering to bring the request to the team, that would be awesome. If you need to contact me directly feel free to do so @ developer.dave@outlook.com. I have reached out to Horizon a few days ago so maybe my name and email are familiar! Thanks again, Dave |
Ether.js obtains the address of a deployed contract by computing it using the sender address + nonce, on EOAs this nonce is globally shared and increases with any transaction, but on Sequence the nonce is exclusive to contract creations.
One possible fix would be to modify the contract creation transaction receipts in such a way that their nonce matches the contract creation nonce. This could be difficult to achieve because the 'contract creation nonce' it's not easily trackeable.
A simpler alternative would be to use a different nonce space for contract creations
The text was updated successfully, but these errors were encountered: