fix: checking for keystore file existence (#2427)

waku-org · Feb 15, 2024 · 8f487a2 · 8f487a2
1 parent 22026b7
commit 8f487a2
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/tests/waku_rln_relay/test_rln_group_manager_onchain.nim b/tests/waku_rln_relay/test_rln_group_manager_onchain.nim
@@ -223,6 +223,13 @@ suite "Onchain group manager":
                                        rlnInstance: manager.rlnInstance)
     expect(ValueError): await manager2.init()
 
+  asyncTest "should error when keystore path and password are provided but file doesn't exist":
+    let manager = await setup()
+    manager.keystorePath = some("/inexistent/file")
+    manager.keystorePassword = some("password")
+
+    expect(CatchableError): await manager.init()
+
   asyncTest "startGroupSync: should start group sync":
     let manager = await setup()
 

diff --git a/waku/waku_rln_relay/group_manager/on_chain/group_manager.nim b/waku/waku_rln_relay/group_manager/on_chain/group_manager.nim
@@ -4,6 +4,7 @@ else:
   {.push raises: [].}
 
 import
+  os,
   web3,
   web3/ethtypes,
   eth/keys as keys,
@@ -630,6 +631,10 @@ method init*(g: OnchainGroupManager): Future[void] {.async.} =
   g.registryContract = some(registryContract)
 
   if g.keystorePath.isSome() and g.keystorePassword.isSome():
+    if not existsFile(g.keystorePath.get()):
+      error "File provided as keystore path does not exist", path=g.keystorePath.get() 
+      raise newException(CatchableError, "missing keystore")
+
     var keystoreQuery = KeystoreMembership(
       membershipContract: MembershipContract(
         chainId: $g.chainId.get(),