Skip to content

EthConnector: add view function to check whether the proof is used.#120

Merged
artob merged 4 commits intodevelopfrom
eth-connector-add-is-used-proof-method
Jun 5, 2021
Merged

EthConnector: add view function to check whether the proof is used.#120
artob merged 4 commits intodevelopfrom
eth-connector-add-is-used-proof-method

Conversation

@sept-en
Copy link
Copy Markdown
Contributor

@sept-en sept-en commented Jun 3, 2021

This feature was requested for both the aurora-frontend and the event-relayer components. The solution seems to be most solid and convenient. IMO this doesn't expose any vulnerability.

@sept-en sept-en added C-enhancement Category: New feature or request P-critical Priority: critical tests labels Jun 3, 2021
@sept-en sept-en changed the base branch from master to develop June 3, 2021 00:39
Comment thread src/parameters.rs
Comment thread src/lib.rs
Copy link
Copy Markdown
Member

@birchmd birchmd left a comment

Choose a reason for hiding this comment

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

LGTM! Nice work.

Comment thread src/parameters.rs
@mfornet mfornet self-requested a review June 3, 2021 16:16
Copy link
Copy Markdown
Contributor

@artob artob left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Good work!

@artob artob merged commit d8f042d into develop Jun 5, 2021
@artob artob deleted the eth-connector-add-is-used-proof-method branch June 5, 2021 00:05
artob added a commit that referenced this pull request Jun 5, 2021
* Remove the dependency on Lunarity. (#115)
* Fix Clippy complaint with +nightly. (#117)
* Add an `evm-bully=yes` build to CI. (#121)
* Fix evm-bully builds. (#130)
* Handle transaction gas limit properly. (#123)
* Add an `is_used_proof` interface. (#120)
* Simplify the `sdk::read_u64` return type. (#118)

Co-authored-by: Frank Braun <frank@near.org>
Co-authored-by: Joshua J. Bouw <dev@joshuajbouw.com>
Co-authored-by: Kirill Abramov <septengineering@pm.me>
Co-authored-by: Michael Birch <michael@near.org>
artob added a commit that referenced this pull request Jun 17, 2021
* Introduce precompiles for the ETH & ERC-20 connectors. (#51)
* Implement generational storage with `SELFDESTRUCT` tests. (#84)
* Remove the dependency on Lunarity. (#115)
* Fix Clippy complaint with `+nightly`. (#117)
* Simplify the `sdk::read_u64` return type. (#118)
* Add an `is_used_proof` interface. (#120)
* Add an `evm-bully=yes` build to CI. (#121)
* Handle transaction gas limit properly. (#123)
* Fix u128 JSON parsing & tests in the ETH connector. (#125)
* Fix evm-bully builds. (#130)
* Add JSON custom error types. (#131)
* Don't burn NEP-141 on deposit. (#133)
* Fix needless borrows. (#135)
* Improve and refactor the ETH connector. (#136)
* Add a macro for logging. (#142)

Co-authored-by: Aleksey Kladov <aleksey@near.org>
Co-authored-by: Arto Bendiken <arto@aurora.dev>
Co-authored-by: Evgeny Ukhanov <evgeny@aurora.dev>
Co-authored-by: Frank Braun <frank@aurora.dev>
Co-authored-by: Joshua J. Bouw <joshua@aurora.dev>
Co-authored-by: Kirill <kirill@aurora.dev>
Co-authored-by: Marcelo Fornet <marcelo@aurora.dev>
Co-authored-by: Michael Birch <michael@aurora.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-enhancement Category: New feature or request P-critical Priority: critical

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants