use provider for permission check

citizenwallet · Nov 10, 2024 · ca0b16e · ca0b16e
1 parent d88e275
commit ca0b16e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/bundler/index.ts b/src/bundler/index.ts
@@ -447,7 +447,7 @@ export class BundlerService {
 
       return hash;
     } catch (e) {
-      if (!(await hasRole(tokenAddress, MINTER_ROLE, from, signer))) {
+      if (!(await hasRole(tokenAddress, MINTER_ROLE, from, this.provider))) {
         throw new Error(
           `Signer (${from}) does not have the MINTER_ROLE on token contract ${tokenAddress}`
         );

diff --git a/src/utils/crypto.ts b/src/utils/crypto.ts
@@ -1,4 +1,4 @@
-import { ethers } from "ethers";
+import { ethers, JsonRpcProvider } from "ethers";
 import accessControlABI from "../abi/IAccessControlUpgradeable.abi.json" with { type: "json" };
 
 export const MINTER_ROLE =
@@ -15,12 +15,12 @@ export async function hasRole(
   tokenAddress: string,
   role: string,
   account: string,
-  signer: ethers.Signer
+  provider: JsonRpcProvider
 ): Promise<boolean> {
   const tokenContract = new ethers.Contract(
     tokenAddress,
     accessControlABI,
-    signer
+    provider
   );
   return await tokenContract.getFunction("hasRole")(role, account);
 }