You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The original implementation was deleting the addresses associated with the contract name just after setting it.
Due to that, it was practically difficult to upgrade the contracts.
Mitigation
PR #32
The mitigation resolved the issue and I confirm the original issue does not exist anymore.
Test
There are new tests added to confirm the upgradability with the same names and all are passing.
Conclusion
LGTM
The text was updated successfully, but these errors were encountered:
C4 issue
M-02: Coding logic of the contract upgrading renders upgrading contracts impractical
Comments
The original implementation was deleting the addresses associated with the contract name just after setting it.
Due to that, it was practically difficult to upgrade the contracts.
Mitigation
PR #32
The mitigation resolved the issue and I confirm the original issue does not exist anymore.
Test
There are new tests added to confirm the upgradability with the same names and all are passing.
Conclusion
LGTM
The text was updated successfully, but these errors were encountered: