fix: send to l2 task incorrectly setting sender (#922)

* fix: send to l2 task incorrectly setting sender

Signed-off-by: Tomás Migone <[email protected]>

Author: tmigone

packages/contracts/tasks/bridge/to-l2.ts

@@ -5,7 +5,10 @@ import { SignerWithAddress } from '@nomiclabs/hardhat-ethers/signers'
 
 greTask('bridge:send-to-l2', 'Bridge GRT tokens from L1 to L2')
   .addParam('amount', 'Amount of tokens to bridge')
-  .addOptionalParam('sender', 'Address of the sender. L1 deployer if empty.')
+  .addOptionalParam(
+    'sender',
+    'Address of the sender, must be managed by the provider node. L1 deployer if empty.',
+  )
   .addOptionalParam('recipient', 'Receiving address in L2. Same to L1 address if empty.')
   .addOptionalParam(
     'deploymentFile',
@@ -22,14 +25,10 @@ greTask('bridge:send-to-l2', 'Bridge GRT tokens from L1 to L2')
     }
 
     // Get the sender, use L1 deployer if not provided
-    const l1Deployer = await graph.l1.getDeployer()
-    const sender: string = taskArgs.sender ?? l1Deployer.address
-
-    const signer = await SignerWithAddress.create(graph.l1.provider.getSigner(sender))
-    if (!signer) {
-      throw new Error(`No wallet found for address ${sender}`)
-    }
-    console.log(`> Using wallet ${signer.address}`)
+    const sender = taskArgs.sender
+      ? await SignerWithAddress.create(graph.l1.provider.getSigner(taskArgs.sender))
+      : await graph.l1.getDeployer()
+    console.log(`> Using wallet ${sender.address}`)
 
     // Patch sendToL2 opts
     taskArgs.l2Provider = graph.l2.provider
@@ -40,7 +39,7 @@ greTask('bridge:send-to-l2', 'Bridge GRT tokens from L1 to L2')
       taskArgs.maxGas = BigNumber.from('400000')
     }
 
-    await sendToL2(graph.contracts, signer, {
+    await sendToL2(graph.contracts, sender, {
       l2Provider: graph.l2.provider,
       amount: taskArgs.amount,
       recipient: taskArgs.recipient,