feat: Ownable2StepUpgradeable to OwnableUpgradeable

[richard-ramos]

This allows the contract to transfer ownership to address 0x000...000 as defined in waku-org/specs#34: At some point, the _Owner_ SHOULD renounce their privileges, and the contract MUST become immutable. The problem with Ownable2StepUpgradeable is that it requires accepting the ownership transfer, which is not possible with address 0x0

[s-tikhomirov]

Would it be possible / desirable to have the contract as Ownable2StepUpgradeable first, and then update it to Ownable? Until the privileges are not dropped, might we need the functionality of transferring the ownership while we depoly and test things? @jm-clius wdyt?

[jm-clius]

No strong opinion, but IIUC the OwnableUpgradable will still allow us to (carefully) transfer ownership if need be, just without the safety of a 2-step process? If we don't have specific plans for multiple ownership transfers I don't think there's a strong need to keep the contract Ownable2StepUpgradeable.

[richard-ramos]

The OwnableUpgradable will still allow us to (carefully) transfer ownership if need be, just without the safety of a 2-step process?

yes! that's basically the main difference between Ownable and Ownable2Step. I do think that changing the ownership of the contract is a process that should happen extremely rarely, with maybe it happening only 2 times, as the owner of the contract itself should not be any individual but perhaps a multisig instead (so changes in the contract are done with approvals of more than a single person). Based on this idea, the first ownership transfer should be to the multisig address, and the final transfer to 0x0 to make it immutable.