emit logs via state-changes in the storage

Author: preston4896

src/automata_pccs/shared/AutomataDaoStorage.sol

@@ -17,6 +17,9 @@ contract AutomataDaoStorage is AutomataTCBManager, IDaoAttestationResolver, Paus
     mapping(address => bool) _authorized_readers;
     mapping(bytes32 attId => bytes collateral) _db;
 
+    event SetAuthorizedWriter(address caller, bool authorized);
+    event SetAuthorizedReader(address caller, bool authorized);
+
     modifier onlyDao(address dao) {
         require(_authorized_writers[dao], "FORBIDDEN");
         _;
@@ -26,15 +29,15 @@ contract AutomataDaoStorage is AutomataTCBManager, IDaoAttestationResolver, Paus
         _initializeOwner(msg.sender);
 
         // adding address(0) as an authorized_reader to allow eth_call
-        _authorized_readers[address(0)] = true;
+        _setAuthorizedReader(address(0), true);
     }
 
     function isAuthorizedCaller(address caller) external view returns (bool) {
         return _authorized_readers[caller];
     }
 
     function setCallerAuthorization(address caller, bool authorized) external onlyOwner {
-        _authorized_readers[caller] = authorized;
+        _setAuthorizedReader(caller, authorized);
     }
 
     function pauseCallerRestriction() external onlyOwner whenNotPaused {
@@ -46,11 +49,11 @@ contract AutomataDaoStorage is AutomataTCBManager, IDaoAttestationResolver, Paus
     }
 
     function grantDao(address granted) external onlyOwner {
-        _authorized_writers[granted] = true;
+        _setAuthorizedWriter(granted, true);
     }
 
     function revokeDao(address revoked) external onlyOwner {
-        _authorized_writers[revoked] = false;
+        _setAuthorizedWriter(revoked, false);
     }
 
     function collateralPointer(bytes32 key) external pure override returns (bytes32 collateralAttId) {
@@ -99,6 +102,16 @@ contract AutomataDaoStorage is AutomataTCBManager, IDaoAttestationResolver, Paus
         attestationId = keccak256(abi.encodePacked(magic, key));
     }
 
+    function _setAuthorizedWriter(address caller, bool authorized) private {
+        _authorized_writers[caller] = authorized;
+        emit SetAuthorizedWriter(caller, authorized);
+    }
+
+    function _setAuthorizedReader(address caller, bool authorized) private {
+        _authorized_readers[caller] = authorized;
+        emit SetAuthorizedReader(caller, authorized);
+    }
+
     /// TCB Management
     using EnumerableSet for EnumerableSet.Bytes32Set;