Improve the AddressResolver to improve protocol upgrades, reduce gas of deployments and further decentralization.
Reduce the complexity of Synthetix contracts using the AddressResolver in three ways:
\nMixinResolver to be given the immutable ReadProxyAddressResolver and no longer require Owned functionalityMixinResolver to have their caches rebuilt by anyoneAddressResolver a function to batch and forward requests to build caches of a collection of MixinResolver contractsThere are currently three main issues necessitating this refactor:
\nFirst, the protocolDAO upgrades are cumbersome, having a plethora of cache rebuilding transactions to be invoked individually. Better to batch these where possible.
\nSecond, as ReadProxyAddressResolver is now available and immutable, contracts can rely on it being set at construction and never changed, and ownership can be remove for contracts that only have it for the MixinResolver.
Third, as many contracts also need to work on the OVM, we need to keep contracts down in size. As the OVM transpiler adds extra code around opcodes like SSTORE, we want to reduce the use of them in constructors as much as possible.
Finally, more emitted events help determine issues by leaving a paper trail of actions performed to be easily traced back.
\nAddressResolver.importAddresses() to emit an event AddressImported for each address addedAddressResolver.rebuildCaches() is exposed (and non-protected), for ease-of useMixinResolver no longer requires Owned as the resolver property is set once only on construction (with the expectation that ReadProxyAddressResolver will be used.MixinResolver has a predicate view isResolverCached()MixinResolver has a rebuildCache() function (publicly available) for the contract to rebuild its cache. This function also emits a CacheUpdated event for each item added to its private cacheMixinResolver.requireAndGetAddress() uses abi.encodePacked for string concatenation and a simpler API.MixinResolver no longer performs tasks in the constructor. Contracts using the mixin must implement the abstract resolverAddressesRequired function.This approach mimics the same functionality as currently on-chain, and does not add any more gas to Synthetix user transactions.
\ncontract AddressResolver {\n\n // restricted function\n function importAddresses(bytes32[] calldata names, address[] calldata destinations) external // ... emits AddressImported\n\n // public function\n function rebuildCaches(MixinResolver[] calldata destinations) external // ... invokes MixinResolver.rebuildCache()\n\n // event\n event AddressImported(bytes32 name, address destination);\n}\ncontract MixinResolver {\n\n // public function\n function rebuildCache() external // ... emits CacheUpdated\n\n // abstract view\n function resolverAddressesRequired() public view returns (bytes32[] memory addresses);\n\n // view\n function isResolverCached() external view returns (bool) // ...\n\n // event\n event CacheUpdated(bytes32 name, address destination);\n}\nSee PR#904
\nN/A
"}},"pageContext":{"id":"41b72780-1bac-541d-99f7-86a116c0f6de","frontmatter__sip":100,"__params":{"frontmatter__sip":"100"}}},"staticQueryHashes":[]}