Grant roles or owner using constructor arguments about contracts-wizard HOT 4 OPEN

A question to ask is whether each role should have its own constructor argument, or if we add just one argument and reuse it for all roles. A third option could be to add one argument for the admin and a separate argument for all other roles.

If the address passed in is zero, the role shouldn't be granted.

from contracts-wizard.

This should also apply for owner if using Ownable.

from contracts-wizard.

This issue is important because if the contract is deployed using CREATE2 or via the "CREATE3" method, msg.sender's value is the address of the factory contract or a temporary proxy contract, instead of the expected account initiating the deployment. Then the factory contract ends up owning the deployed contract and receiving initially minted tokens (that then may be stuck forever).

from contracts-wizard.

tx.origin gives the expected value but Vitalik said it shouldn't be relied upon.

from contracts-wizard.