@etherisc/gif-next 0.0.2-952752e-374 → 0.0.2-957bd35-589

package/contracts/instance/InstanceAdmin.sol

@@ -1,81 +1,49 @@
 // SPDX-License-Identifier: UNLICENSED
 pragma solidity ^0.8.20;
 
-
-import {AccessAdmin} from "../authorization/AccessAdmin.sol";
-import {AccessManagerCloneable} from "../authorization/AccessManagerCloneable.sol";
-import {IAuthorization} from "../authorization/IAuthorization.sol";
-import {IInstanceLinkedComponent} from "../shared/IInstanceLinkedComponent.sol";
+import {IAccess} from "../authorization/IAccess.sol";
 import {IAuthorization} from "../authorization/IAuthorization.sol";
 import {IRegistry} from "../registry/IRegistry.sol";
 import {IInstance} from "./IInstance.sol";
-import {ObjectType} from "../type/ObjectType.sol";
-import {RoleId, RoleIdLib} from "../type/RoleId.sol";
-import {Str, StrLib} from "../type/String.sol";
-import {TokenHandler} from "../shared/TokenHandler.sol";
-import {VersionPart, VersionPartLib} from "../type/Version.sol";
+
+import {AccessAdmin} from "../authorization/AccessAdmin.sol";
+import {AccessAdminLib} from "../authorization/AccessAdminLib.sol";
+import {AccessManagerCloneable} from "../authorization/AccessManagerCloneable.sol";
+import {ObjectType, INSTANCE} from "../type/ObjectType.sol";
+import {RoleId, ADMIN_ROLE} from "../type/RoleId.sol";
+import {Str} from "../type/String.sol";
+import {VersionPart} from "../type/Version.sol";
+import {INSTANCE_TARGET_NAME, INSTANCE_ADMIN_TARGET_NAME, INSTANCE_STORE_TARGET_NAME, PRODUCT_STORE_TARGET_NAME, BUNDLE_SET_TARGET_NAME, RISK_SET_TARGET_NAME} from "./TargetNames.sol";
 
 
 contract InstanceAdmin is
     AccessAdmin
 {
-    string public constant INSTANCE_TARGET_NAME = "Instance";
-    string public constant INSTANCE_STORE_TARGET_NAME = "InstanceStore";
-    string public constant INSTANCE_ADMIN_TARGET_NAME = "InstanceAdmin";
-    string public constant BUNDLE_SET_TARGET_NAME = "BundleSet";
-    string public constant RISK_SET_TAGET_NAME = "RiskSet";
-
-    uint64 public constant CUSTOM_ROLE_ID_MIN = 10000; // MUST be even
+    // onlyInstanceService
+    error ErrorInstanceAdminNotInstanceService(address caller);
 
-    error ErrorInstanceAdminCallerNotInstanceOwner(address caller);
-    error ErrorInstanceAdminInstanceAlreadyLocked();
-    error ErrorInstanceAdminNotRegistered(address target);
-    error ErrorInstanceAdminAlreadyAuthorized(address target);
-    error ErrorInstanceAdminReleaseMismatch();
-    error ErrorInstanceAdminExpectedTargetMissing(string targetName);
+    // authorizeFunctions
+    error ErrorInstanceAdminNotComponentOrCustomTarget(address target);
 
     IInstance internal _instance;
     IRegistry internal _registry;
-    uint64 internal _idNext;
 
-    IAuthorization internal _instanceAuthorization;
+    uint64 internal _customRoleIdNext;
 
 
-    modifier onlyInstanceOwner() {        
-        if(msg.sender != _registry.ownerOf(address(_instance))) {
-            revert ErrorInstanceAdminCallerNotInstanceOwner(msg.sender);
+    modifier onlyInstanceService() {
+        if (msg.sender != _registry.getServiceAddress(INSTANCE(), getRelease())) {
+            revert ErrorInstanceAdminNotInstanceService(msg.sender);
         }
         _;
     }
 
-    /// @dev Only used for master instance admin.
-    /// Contracts created via constructor come with disabled initializers.
-    constructor(
-        address instanceAuthorization
-    ) {
-        initialize(new AccessManagerCloneable());
-
-        _instanceAuthorization = IAuthorization(instanceAuthorization);
-
-        _disableInitializers();
-    }
-
-
-    function initialize(
-        AccessManagerCloneable clonedAccessManager,
-        IRegistry registry,
-        VersionPart release
-    )
-        external
-        initializer()
-    {
-        __AccessAdmin_init(clonedAccessManager);
-
-        clonedAccessManager.completeSetup(
-            address(registry), 
-            release); 
 
-        _registry = registry;
+    /// @dev Only used for master instance admin.
+    constructor(address accessManager) {
+        initialize(
+            accessManager,
+            "MasterInstanceAdmin");
     }
 
 
@@ -83,96 +51,138 @@ contract InstanceAdmin is
     /// Important: Initialization of instance admin is only complete after calling this function. 
     /// Important: The instance MUST be registered and all instance supporting contracts must be wired to this instance.
     function completeSetup(
-        address instance,
-        address authorization
+        address registry,
+        address authorization,
+        VersionPart release,
+        address instance
     )
         external
-        reinitializer(uint64(getRelease().toInt()))
-        onlyDeployer()
+        reinitializer(uint64(release.toInt()))
     {
-        _idNext = CUSTOM_ROLE_ID_MIN;
+        // checks
+        AccessAdminLib.checkIsRegistered(registry, instance, INSTANCE());
+
+        // effects
+        AccessManagerCloneable(
+            authority()).completeSetup(
+                registry, 
+                release); 
+
+        AccessAdminLib.checkAuthorization(
+            address(_authorization), 
+            authorization, 
+            INSTANCE(), // expectedDomain
+            release, // expectedRelease
+            false, // expectServiceAuthorization
+            true); // checkAlreadyInitialized
+
+        _registry = IRegistry(registry);
         _instance = IInstance(instance);
-        _instanceAuthorization = IAuthorization(authorization);
+        _authorization = IAuthorization(authorization);
+        _customRoleIdNext = 0;
 
-        // AccessManagerCloneable accessManager = AccessManagerCloneable(authority());
-        // accessManager.completeSetup(
-        //     address(_registry), 
-        //     release); 
+        // link nft ownability to instance
+        _linkToNftOwnable(instance);
 
-        _checkTargetIsReadyForAuthorization(instance);
+        // setup roles and services
+        _createRoles(_authorization);
+        _setupServices(_authorization);
 
-        // check matching releases
-        if (_instanceAuthorization.getRelease() != getRelease()) {
-            revert ErrorInstanceAdminReleaseMismatch();
+        // setup instance targets
+        _createInstanceTargets(_authorization.getMainTargetName());
+
+        // authorize functions of instance contracts
+        _createTargetAuthorizations(_authorization);
+    }
+
+    /// @dev grants the service roles to the service addresses based on the authorization specification.
+    /// Service addresses used for the granting are determined by the registry and the release of this instance.
+    function _setupServices(IAuthorization authorization)
+        internal
+    {
+        ObjectType[] memory serviceDomains = authorization.getServiceDomains();
+
+        for(uint256 i = 0; i < serviceDomains.length; i++) {
+            ObjectType serviceDomain = serviceDomains[i];
+            RoleId serviceRoleId = authorization.getServiceRole(serviceDomain);
+            address service = _registry.getServiceAddress(serviceDomain, getRelease());
+
+            _grantRoleToAccount(
+                serviceRoleId,
+                service);
         }
+    }
 
-        // add instance authorization
-        _createRoles(_instanceAuthorization);
-        _createModuleTargetsWithRoles();
-        _createTargetAuthorizations(_instanceAuthorization);
+
+    function _createInstanceTargets(string memory instanceTargetName)
+        internal
+    {
+        _createInstanceTarget(address(_instance), instanceTargetName); 
+        _createInstanceTarget(address(this), INSTANCE_ADMIN_TARGET_NAME); 
+        _createInstanceTarget(address(_instance.getInstanceStore()), INSTANCE_STORE_TARGET_NAME); 
+        _createInstanceTarget(address(_instance.getProductStore()), PRODUCT_STORE_TARGET_NAME); 
+        _createInstanceTarget(address(_instance.getBundleSet()), BUNDLE_SET_TARGET_NAME); 
+        _createInstanceTarget(address(_instance.getRiskSet()), RISK_SET_TARGET_NAME); 
+    }
+
+
+    function _createInstanceTarget(address target, string memory name) internal {
+        _createTarget(target, name, TargetType.Instance, true); 
     }
 
 
     /// @dev Initializes the authorization for the specified component.
     /// Important: The component MUST be registered.
     function initializeComponentAuthorization(
-        IInstanceLinkedComponent component
+        address componentAddress,
+        ObjectType expectedType
     )
         external
+        restricted()
     {
-        // !!! TODO add caller restrictions?
+        IAuthorization authorization = AccessAdminLib.checkComponentInitialization(
+            this, _authorization, componentAddress, expectedType);
 
-        _checkTargetIsReadyForAuthorization(address(component));
+        // effects
+        _createRoles(authorization);
+        _createTarget(componentAddress, authorization.getMainTargetName(), TargetType.Component, true);
+        _createTargetAuthorizations(authorization);
+    }
 
-        // get authorization specification
-        IAuthorization authorization = component.getAuthorization();
-        string memory targetName = authorization.getTargetName();
-        _checkTargetIsReadyForAuthorization(address(component));
 
+    /// @dev Creates a custom role.
+    function createRole(
+        string memory name,
+        RoleId adminRoleId,
+        uint32 maxMemberCount
+    )
+        external
+        restricted()
+        returns (RoleId roleId)
+    {
+        // create roleId
+        roleId = AccessAdminLib.getCustomRoleId(_customRoleIdNext++);
+
+        // create role
+        _createRole(
+            roleId, 
+            AccessAdminLib.roleInfo(
+                adminRoleId, 
+                IAccess.TargetType.Custom, 
+                maxMemberCount, 
+                name),
+            true); // revert on existing role
+    }
 
-        // create roles
-        _createRoles(authorization);
 
-        // create component target
-        _createTarget(
-            address(component), 
-            targetName, 
-            true, // checkAuthority
-            false); // custom
-
-        _createTarget(
-            address(component.getTokenHandler()), 
-            string(abi.encodePacked(targetName, "TH")), 
-            true, 
-            false);
-        
-        FunctionInfo[] memory functions = new FunctionInfo[](3);
-        functions[0] = toFunction(TokenHandler.collectTokens.selector, "collectTokens");
-        functions[1] = toFunction(TokenHandler.collectTokensToThreeRecipients.selector, "collectTokensToThreeRecipients");
-        functions[2] = toFunction(TokenHandler.distributeTokens.selector, "distributeTokens");
-
-        // FIXME: make this a bit nicer and work with IAuthorization. Use a specific role, not public - access to TokenHandler must be restricted
-        _authorizeTargetFunctions(
-            address(component.getTokenHandler()),
-            getPublicRole(),
-            functions);
-
-        _grantRoleToAccount(
-            authorization.getTargetRole(
-                authorization.getMainTarget()), 
-            address(component));
-        
-        _createTargetAuthorizations(authorization);
+    /// @dev Activtes/pauses the specified role.
+    function setRoleActive(RoleId roleId, bool active)
+        external
+        restricted()
+    {
+        _setRoleActive(roleId, active);
     }
 
-    /// @dev Creates a custom role
-    // TODO implement
-    // function createRole()
-    //     external
-    //     restricted()
-    // {
-
-    // }
 
     /// @dev Grants the provided role to the specified account
     function grantRole(
@@ -185,70 +195,99 @@ contract InstanceAdmin is
     }
 
 
-    function setInstanceLocked(bool locked)
+    /// @dev Revokes the provided role from the specified account
+    function revokeRole(
+        RoleId roleId, 
+        address account)
         external
-        onlyInstanceOwner()
+        restricted()
     {
-        AccessManagerCloneable accessManager = AccessManagerCloneable(authority());
-
-        if(accessManager.isLocked() == locked) {
-            revert ErrorInstanceAdminInstanceAlreadyLocked();
-        }
-        accessManager.setLocked(locked);
+        _revokeRoleFromAccount(roleId, account);
     }
 
-    function setTargetLocked(address target, bool locked) 
-        external 
+
+    /// @dev Create a new contract target.
+    /// The target needs to be an access managed contract.
+    function createTarget(
+        address target, 
+        string memory name
+    )
+        external
         restricted()
+        returns (RoleId contractRoleId)
     {
-        _setTargetClosed(target, locked);
+        return _createTarget(
+            target, 
+            name, 
+            TargetType.Contract,
+            true); // check authority matches
     }
 
 
+    /// @dev Add function authorizations for the specified component or custom target.
+    function authorizeFunctions(
+        address target,
+        RoleId roleId,
+        IAccess.FunctionInfo[] memory functions
+    )
+        external
+        restricted()
+    {
+        _authorizeTargetFunctions(target, roleId, functions, true, true);
+    }
 
-    /// @dev Returns the instance authorization specification used to set up this instance admin.
-    function getInstanceAuthorization()
+
+    /// @dev Removes function authorizations for the specified component or custom target.
+    function unauthorizeFunctions(
+        address target,
+        IAccess.FunctionInfo[] memory functions
+    )
         external
-        view
-        returns (IAuthorization instanceAuthorizaion)
+        restricted()
     {
-        return _instanceAuthorization;
+        _authorizeTargetFunctions(target, ADMIN_ROLE(), functions, true, false);
     }
 
-    // ------------------- Internal functions ------------------- //
 
-    function _checkTargetIsReadyForAuthorization(address target)
-        internal
-        view
+    /// @dev locks the instance and all its releated targets including component and custom targets.
+    function setInstanceLocked(bool locked)
+        external
+        // not restricted(): need to operate on locked instances to unlock instance
+        onlyInstanceService()
     {
-        if (!_registry.isRegistered(target)) {
-            revert ErrorInstanceAdminNotRegistered(target);
-        }
+        AccessManagerCloneable accessManager = AccessManagerCloneable(authority());
+        accessManager.setLocked(locked);
+    }
 
-        if (targetExists(target)) {
-            revert ErrorInstanceAdminAlreadyAuthorized(target);
-        }
+
+    function setTargetLocked(address target, bool locked) 
+        external 
+        // not restricted(): need to operate on locked instances to unlock instance
+        onlyInstanceService()
+    {
+        _setTargetLocked(target, locked);
     }
 
 
-    function _createRoles(IAuthorization authorization)
-        internal
+    function setContractLocked(address target, bool locked) 
+        external 
+        restricted() // component service
     {
-        RoleId[] memory roles = authorization.getRoles();
-        RoleId roleId;
-        RoleInfo memory roleInfo;
+        _setTargetLocked(target, locked);
+    }
 
-        for(uint256 i = 0; i < roles.length; i++) {
-            roleId = roles[i];
 
-            if (!roleExists(roleId)) {
-                _createRole(
-                    roleId,
-                    authorization.getRoleInfo(roleId));
-            }
-        }
+    /// @dev Returns the instance authorization specification used to set up this instance admin.
+    function getInstanceAuthorization()
+        external
+        view
+        returns (IAuthorization instanceAuthorizaion)
+    {
+        return _authorization;
     }
 
+    // ------------------- Internal functions ------------------- //
+
 
     function _createTargetAuthorizations(IAuthorization authorization)
         internal
@@ -259,68 +298,10 @@ contract InstanceAdmin is
         for(uint256 i = 0; i < targets.length; i++) {
             target = targets[i];
             RoleId[] memory authorizedRoles = authorization.getAuthorizedRoles(target);
-            RoleId authorizedRole;
 
             for(uint256 j = 0; j < authorizedRoles.length; j++) {
-                authorizedRole = authorizedRoles[j];
-
-                _authorizeTargetFunctions(
-                    getTargetForName(target),
-                    authorizedRole,
-                    authorization.getAuthorizedFunctions(
-                        target, 
-                        authorizedRole));
+                _authorizeFunctions(authorization, target, authorizedRoles[j]);
             }
         }
     }
-
-    function _checkAndCreateTargetWithRole(
-        address target,
-        string memory targetName
-    )
-        internal
-    {
-        // check that target name is defined in authorization specification
-        Str name = StrLib.toStr(targetName);
-        if (!_instanceAuthorization.targetExists(name)) {
-            revert ErrorInstanceAdminExpectedTargetMissing(targetName);
-        }
-
-        // create named target
-        _createTarget(
-            target, 
-            targetName, 
-            false, // check authority TODO check normal targets, don't check service targets (they share authority with release admin)
-            false);
-
-        // assign target role if defined
-        RoleId targetRoleId = _instanceAuthorization.getTargetRole(name);
-        if (targetRoleId != RoleIdLib.zero()) {
-            _grantRoleToAccount(targetRoleId, target);
-        }
-    }
-
-    function _createModuleTargetsWithRoles()
-        internal
-    {
-        // create module targets
-        _checkAndCreateTargetWithRole(address(_instance), INSTANCE_TARGET_NAME);
-        _checkAndCreateTargetWithRole(address(_instance.getInstanceStore()), INSTANCE_STORE_TARGET_NAME);
-        _checkAndCreateTargetWithRole(address(_instance.getInstanceAdmin()), INSTANCE_ADMIN_TARGET_NAME);
-        _checkAndCreateTargetWithRole(address(_instance.getBundleSet()), BUNDLE_SET_TARGET_NAME);
-        _checkAndCreateTargetWithRole(address(_instance.getRiskSet()), RISK_SET_TAGET_NAME);
-
-        // create targets for services that need to access the module targets
-        ObjectType[] memory serviceDomains = _instanceAuthorization.getServiceDomains();
-        VersionPart release = _instanceAuthorization.getRelease();
-        ObjectType serviceDomain;
-
-        for (uint256 i = 0; i < serviceDomains.length; i++) {
-            serviceDomain = serviceDomains[i];
-
-            _checkAndCreateTargetWithRole(
-                _registry.getServiceAddress(serviceDomain, release),
-                _instanceAuthorization.getServiceTarget(serviceDomain).toString());
-        }
-    }
 }