fix(payloads): fix payload format for relay-signer (#604)

* fix(payloads): fix payload format * test(relay-signer): fix tests
OpenZeppelin · Oct 29, 2024 · 719ab30 · 719ab30
1 parent 82d3b74
commit 719ab30
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/packages/relay-signer/src/action/index.test.ts b/packages/relay-signer/src/action/index.test.ts
@@ -88,7 +88,7 @@ describe('ActionRelayer', () => {
       expect(relayer.lambda.invoke).toHaveBeenCalledWith({
         FunctionName: 'arn',
         InvocationType: 'RequestResponse',
-        Payload: '{"action":"replace-tx","payload":{"to":"0x0","gasLimit":21000}}',
+        Payload: '{"action":"replace-tx","payload":{"to":"0x0","gasLimit":21000,"nonce":10}}',
       });
     });
 
@@ -97,7 +97,7 @@ describe('ActionRelayer', () => {
       expect(relayer.lambda.invoke).toHaveBeenCalledWith({
         FunctionName: 'arn',
         InvocationType: 'RequestResponse',
-        Payload: '{"action":"replace-tx","txPayload":{"to":"0x0","gasLimit":21000,"id":"123-456-abc"}}',
+        Payload: '{"action":"replace-tx","payload":{"to":"0x0","gasLimit":21000,"id":"123-456-abc"}}',
       });
     });
   });
@@ -108,7 +108,7 @@ describe('ActionRelayer', () => {
       expect(relayer.lambda.invoke).toHaveBeenCalledWith({
         FunctionName: 'arn',
         InvocationType: 'RequestResponse',
-        Payload: '{"action":"cancel-tx","id":"123-456-abc"}',
+        Payload: '{"action":"cancel-tx","payload":"123-456-abc"}',
       });
     });
   });

diff --git a/packages/relay-signer/src/action/index.ts b/packages/relay-signer/src/action/index.ts
@@ -54,19 +54,20 @@ export class ActionRelayer extends BaseActionClient implements IRelayer {
 
   public async replaceTransactionById(id: string, payload: RelayerTransactionPayload): Promise<RelayerTransaction> {
     const txPayload: RelayerTransactionPayload & { id: string } = { ...payload, id };
-    return this.execute({ action: 'replace-tx', txPayload });
+    return this.execute({ action: 'replace-tx', payload: txPayload });
   }
 
   public async replaceTransactionByNonce(
     nonce: number,
     payload: RelayerTransactionPayload,
   ): Promise<RelayerTransaction> {
     const txPayload: RelayerTransactionPayload & { nonce: number } = { ...payload, nonce };
-    return this.execute({ action: 'replace-tx', payload });
+    return this.execute({ action: 'replace-tx', payload: txPayload });
   }
 
   public async cancelTransactionById(id: string): Promise<TransactionDeleteResponse> {
-    return this.execute({ action: 'cancel-tx', id });
+    const payload = id;
+    return this.execute({ action: 'cancel-tx', payload });
   }
 
   public async getRelayer(): Promise<RelayerGetResponse> {