-
Notifications
You must be signed in to change notification settings - Fork 78
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
Feat(engine): Include address field in returned EVM logs #299
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets refrain from merging for now. This is a breaking change as realistically we probably won't be able to adequately prepare for this for another 1 - 2 weeks.
After aurora-is-near/aurora.js#22 is merged (and anything that depends on |
Noted. |
@joshuajbouw Is there anything preventing us from merging this PR and releasing it ? |
Of course, API breaking. Everything that depend on the engine requires updating first. |
ee43976
to
6ddab87
Compare
aurora-is-near/aurora.js#22 has now been merged and deployed to all endpoints, so this is good to go (modulo adequate validation). |
Great we'll get this in. @birchmd |
6ddab87
to
45c8edd
Compare
* Feat(engine): Public method for computing block hash at given height (#303) * Fix(precompiles): Always charge for gas in ecrecover (#305) * Fix(precompiles): Pad modexp input if it is too short (#306) * Feat(engine): Include address field in returned EVM logs (#299) * Fix(engine): Remove unnecessary eth-connector logic (#312) * Feat(benchmarks): update gas bounds after wasm cost reduction (#315) * Fix(engine): update to latest SputnikVM (#316) * Add logging of public of the signer (#319) Co-authored-by: Dmitry Strokov <dmitry.strokov@aurora.dev> Co-authored-by: Joshua J. Bouw <joshua@aurora.dev>
In addition to adding the address field to the returned EVM logs, this PR also introduces a version byte to the
SubmitResult
ABI. This byte can be used to discriminate the new ABI from the old, and in the future can discriminate future breaking ABI changes from each other.closes #286