search

yetanotherco / aligned_layer

Aligned is a verification layer for zero-knowledge proofs using EigenLayer. Our mission is to accelerate the adoption of zero-knowledge and validity proofs on Ethereum.
https://alignedlayer.com/
MIT License
145 stars 341 forks source link

BatcherPaymentService use `Ownable2StepUpgradeable` instead of `OwnableUpgradeable` #900

Closed entropidelic closed 3 weeks ago

entropidelic commented 2 months ago

OwnableUpgradeable has a shortcoming that it allows the owner to transfer ownership to a non-existent or mistyped address. This can lead to a loss of control over the contract. Ownable2StepUpgradeable is a more secure alternative.

Reference: Openzeppelin Ownable: Use Ownable2Step Instead

IAvecilla commented 3 weeks ago

We are currently using the OpenZeppelin contracts from eigenlayer-middleware, but the contract with Ownable2StepUpgradeable is not yet available on the Eigenlayer side. To maintain consistency between the contract versions we are using, we should wait for them to add it before address this.