search

polkadot-evm / frontier

Ethereum compatibility layer for Substrate.
Apache License 2.0
574 stars 490 forks source link

Does an incorrect implementation of ConsensusDataProvider pose a risk of vulnerabilities? #1535

Open jyaymusk opened 4 weeks ago

jyaymusk commented 4 weeks ago

Does an incorrect implementation of ConsensusDataProvider pose a risk of vulnerabilities? What is the most serious incident associated with it?

I noticed that there is a lack of BABE consensus-related implementations here, but I found it in a community project. If you have time, I hope you can take a look at its implementation for any potential issues.

community BabeConsensusDataProvider

Also, thank you very much for your contributions to the open-source code, it has saved us a lot of time!

jyaymusk commented 3 weeks ago

In addition, I found the implementation of debug here, but did not see the RPC support of trace. What is the reason? Because I found that moonbean has trace support, I think this is a common demand. Thanks again