fixup: make authorizations public

Author: matiasedgeandnode

packages/horizon/contracts/utilities/Authorizable.sol

@@ -20,7 +20,7 @@ abstract contract Authorizable is IAuthorizable {
     uint256 public immutable REVOKE_AUTHORIZATION_THAWING_PERIOD;
 
     /// @notice Authorization details for authorizer-signer pairs
-    mapping(address signer => Authorization authorization) private _authorizations;
+    mapping(address signer => Authorization authorization) public authorizations;
 
     /**
      * @dev Revert if the caller has not authorized the signer
@@ -43,52 +43,52 @@ abstract contract Authorizable is IAuthorizable {
      */
     function authorizeSigner(address signer, uint256 proofDeadline, bytes calldata proof) external {
         require(
-            _authorizations[signer].authorizer == address(0),
+            authorizations[signer].authorizer == address(0),
             AuthorizableSignerAlreadyAuthorized(
-                _authorizations[signer].authorizer,
+                authorizations[signer].authorizer,
                 signer,
-                _authorizations[signer].revoked
+                authorizations[signer].revoked
             )
         );
         _verifyAuthorizationProof(proof, proofDeadline, signer);
-        _authorizations[signer].authorizer = msg.sender;
+        authorizations[signer].authorizer = msg.sender;
         emit SignerAuthorized(msg.sender, signer);
     }
 
     /**
      * See {IAuthorizable.thawSigner}.
      */
     function thawSigner(address signer) external onlyAuthorized(signer) {
-        _authorizations[signer].thawEndTimestamp = block.timestamp + REVOKE_AUTHORIZATION_THAWING_PERIOD;
-        emit SignerThawing(msg.sender, signer, _authorizations[signer].thawEndTimestamp);
+        authorizations[signer].thawEndTimestamp = block.timestamp + REVOKE_AUTHORIZATION_THAWING_PERIOD;
+        emit SignerThawing(msg.sender, signer, authorizations[signer].thawEndTimestamp);
     }
 
     /**
      * See {IAuthorizable.cancelThawSigner}.
      */
     function cancelThawSigner(address signer) external onlyAuthorized(signer) {
-        require(_authorizations[signer].thawEndTimestamp > 0, AuthorizableSignerNotThawing(signer));
-        uint256 thawEnd = _authorizations[signer].thawEndTimestamp;
-        _authorizations[signer].thawEndTimestamp = 0;
+        require(authorizations[signer].thawEndTimestamp > 0, AuthorizableSignerNotThawing(signer));
+        uint256 thawEnd = authorizations[signer].thawEndTimestamp;
+        authorizations[signer].thawEndTimestamp = 0;
         emit SignerThawCanceled(msg.sender, signer, thawEnd);
     }
 
     /**
      * See {IAuthorizable.revokeAuthorizedSigner}.
      */
     function revokeAuthorizedSigner(address signer) external onlyAuthorized(signer) {
-        uint256 thawEndTimestamp = _authorizations[signer].thawEndTimestamp;
+        uint256 thawEndTimestamp = authorizations[signer].thawEndTimestamp;
         require(thawEndTimestamp > 0, AuthorizableSignerNotThawing(signer));
         require(thawEndTimestamp <= block.timestamp, AuthorizableSignerStillThawing(block.timestamp, thawEndTimestamp));
-        _authorizations[signer].revoked = true;
+        authorizations[signer].revoked = true;
         emit SignerRevoked(msg.sender, signer);
     }
 
     /**
      * See {IAuthorizable.getThawEnd}.
      */
     function getThawEnd(address signer) external view returns (uint256) {
-        return _authorizations[signer].thawEndTimestamp;
+        return authorizations[signer].thawEndTimestamp;
     }
 
     /**
@@ -103,8 +103,8 @@ abstract contract Authorizable is IAuthorizable {
      */
     function _isAuthorized(address _authorizer, address _signer) internal view returns (bool) {
         return (_authorizer != address(0) &&
-            _authorizations[_signer].authorizer == _authorizer &&
-            !_authorizations[_signer].revoked);
+            authorizations[_signer].authorizer == _authorizer &&
+            !authorizations[_signer].revoked);
     }
 
     /**