Changing max gas for revocation list transaction

Author: thiagoromanos

aries_cloudagent/anoncreds/default/did_besu/registry.py

@@ -148,6 +148,7 @@ def __init__(self):
         self.VALIDATOR_CONTROL_ADDRESS = None
         self.ROLE_CONTROL_ADDRESS = None
         self.REVOCATION_ADDRESS = None
+        self.REVOCATION_LIST_GAS_LIMIT = "0x1fffffffffffff"
 
     @property
     def supported_identifiers_regex(self) -> Pattern:
@@ -168,6 +169,10 @@ async def setup(self, context: InjectionContext):
         self.REVOCATION_ADDRESS = context.settings.get(
             "ledger.revocation_contract_address"
         )
+        if context.settings.get("ledger.revocation_list_gas_limit"):
+            self.REVOCATION_LIST_GAS_LIMIT = context.settings.get(
+                "ledger.revocation_list_gas_limit"
+            )
 
         self.web3 = Web3(Web3.HTTPProvider(self.HTTP_PROVIDER))
         self.web3.middleware_onion.inject(geth_poa_middleware, layer=0)
@@ -276,7 +281,13 @@ async def register_revocation(
         Chain_id = self.web3.eth.chain_id
         nonce = self.web3.eth.get_transaction_count(self.ACCOUNT)
         call_function = contract.functions.createRevocation(rev_json).build_transaction(
-            {"chainId": Chain_id, "from": self.ACCOUNT, "nonce": nonce, "gas": 3000000}
+            {
+                "chainId": Chain_id,
+                "from": self.ACCOUNT,
+                "nonce": nonce,
+                "gas": 3000000,
+                "gasPrice": self.web3.eth.gas_price,
+            }
         )
 
         tx_receipt = await self.send_transaction_tx(call_function)
@@ -620,6 +631,7 @@ async def register_revocation_registry_definition(
                     "from": self.ACCOUNT,
                     "nonce": nonce,
                     "gas": 3000000,
+                    "gasPrice": self.web3.eth.gas_price,
                 }
             )
 
@@ -786,7 +798,8 @@ async def _revoc_reg_entry_with_fix(
                     "chainId": Chain_id,
                     "from": self.ACCOUNT,
                     "nonce": nonce,
-                    "gas": "0x1fffffffffffff",
+                    "gas": int(self.REVOCATION_LIST_GAS_LIMIT),
+                    "gasPrice": self.web3.eth.gas_price,
                 }
             )
 

aries_cloudagent/config/argparse.py

@@ -876,6 +876,17 @@ def add_arguments(self, parser: ArgumentParser):
                 "starts with 0x, this prefix is added."
             ),
         )
+        parser.add_argument(
+            "--revocation-list-gas-limit",
+            type=str,
+            metavar="<account>",
+            required=False,
+            dest="revocation_list_gas_limit",
+            env_var="ACAPY_REVOCATION_LIST_GAS_LIMIT",
+            help=(
+                "Specifies de gas limit for the update of revocation list transaction."
+            ),
+        )
         parser.add_argument(
             "--ledger-pool-name",
             type=str,
@@ -1015,6 +1026,10 @@ def get_settings(self, args: Namespace) -> dict:
                 if args.indy_did_contract_address.startswith("0x")
                 else f"0x{args.indy_did_contract_address}"
             )
+            if args.revocation_list_gas_limit:
+                settings["ledger.revocation_list_gas_limit"] = (
+                    args.revocation_list_gas_limit
+                )
         else:
             single_configured = False
             multi_configured = False

aries_cloudagent/ledger/besu_vdr.py

@@ -382,6 +382,7 @@ def _send_signed_transaction(
             "chainId": chain_id,
             "from": self.ledgerConfig.trusteeAccount,
             "nonce": nonce,
+            "gasPrice": self.web3.eth.gas_price,
         }
         if includeGasInTx:
             txParams["gas"] = 3000000