iron-fish / ironfish

A novel cryptocurrency focused on privacy and accessibility.
https://ironfish.network
Mozilla Public License 2.0
964 stars 579 forks source link

adds minimal verification to ironfish transactions for evm #5080

Closed jowparks closed 1 week ago

jowparks commented 1 week ago

Summary

Simple EvmDescription verification via the EVM. This is a roundtrip transaction that passes through rust, to node typescript, and then to EVM.

Simply verifies the transaction is executable. Only form of validation explicitly done is signature for the unit test.

Testing Plan

tests pass

Documentation

Does this change require any updates to the Iron Fish Docs (ex. the RPC API Reference)? If yes, link a related documentation pull request for the website.

[ ] Yes

Breaking Change

Is this a breaking change? If yes, add notes below on why this is breaking and label it with breaking-change-rpc or breaking-change-sdk.

[ ] Yes