ChainSafe / chainbridge-celo

GNU Lesser General Public License v3.0
3 stars 2 forks source link

Fix SignatureVerification signature param #102

Closed P1sar closed 3 years ago

P1sar commented 3 years ago

SignatureVerification params that are used to call bridge smart contract method execute are malformed now need to be fixed.

Implementation details

Signature seems to be located at extra.AggregatedSeal.Signature. https://github.com/ChainSafe/chainbridge-celo/blob/main/chain/listener/listener.go#L217 https://github.com/ChainSafe/chainbridge-celo/blob/main/chain/listener/listener.go#L204 https://github.com/celo-org/celo-blockchain/blob/8cfe3d15966eb99deb80ba82686230ca588e91dd/consensus/istanbul/backend/engine.go#L263

Testing details

Unit tests

Acceptance Criteria