{"componentChunkName":"component---src-pages-sips-sip-markdown-remark-frontmatter-sip-tsx","path":"/sips/sip-235/","result":{"data":{"markdownRemark":{"fileAbsolutePath":"/vercel/path0/content/sips/sip-235.md","frontmatter":{"sip":235,"sccp":null,"title":"Update Issuer Modifier","network":"Ethereum & Optimism","author":"MEB (@barrasso)","type":"Governance","proposal":"https://snapshot.org/#/snxgov.eth/proposal/QmPRtghu9EnPedgsAZrLoWGyw9PiNkjjvCowox68LFh2wh","implementor":"MEB (@barrasso)","release":"Saiph","created":"2022-05-11T00:00:00.000Z","updated":null,"status":"Implemented"},"html":"

Simple Summary

\n\n

Adds another require check to the onlyTrustedMinters modifier in Issuer.

\n

Abstract

\n\n

On each network, only one of those bridge addresses is the trusted contract and the other address has to be 0. Adding this check to the modifier will greatly increase the overall security of the system going forward.

\n

Motivation

\n\n

To increase the security of this access control for future deployments.

\n

Technical Specification

\n

Issuer.onlyTrustedMinters

\n

We should patch this modifier to either only check the correct address for the chain it's on, or to check that one of the addresses == address(0).

\n
modifier onlyTrustedMinters() {\n    require(\n      msg.sender == resolver.getAddress(CONTRACT_SYNTHETIXBRIDGETOOPTIMISM) ||\n      msg.sender == resolver.getAddress(CONTRACT_SYNTHETIXBRIDGETOBASE),\n      \"Issuer: Only trusted minters can perform this action\"\n    );\n    require(\n      resolver.getAddress(CONTRACT_SYNTHETIXBRIDGETOOPTIMISM) == address(0) ||\n      resolver.getAddress(CONTRACT_SYNTHETIXBRIDGETOBASE) == address(0),\n      \"Issuer: One of the trusted minters is not 0\"\n    );\n    _;\n}\n
\n

Configurable Values (Via SCCP)

\n

N/A

\n

Copyright

\n

Copyright and related rights waived via CC0.

"}},"pageContext":{"id":"e67bf206-2747-5c6a-8e1c-77d1b77ff127","frontmatter__sip":235,"__params":{"frontmatter__sip":"235"}}},"staticQueryHashes":[]}