From b1184b5ac8e1fa3666176573173a7525c8a8463d Mon Sep 17 00:00:00 2001 From: 0xPatrick Date: Mon, 25 Mar 2024 12:49:03 -0400 Subject: [PATCH] fix: ConnectionHandler methods should return promises --- .../orchestration/src/contracts/stakeAtom.contract.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/orchestration/src/contracts/stakeAtom.contract.js b/packages/orchestration/src/contracts/stakeAtom.contract.js index 55017228e13..c7f35260576 100644 --- a/packages/orchestration/src/contracts/stakeAtom.contract.js +++ b/packages/orchestration/src/contracts/stakeAtom.contract.js @@ -34,17 +34,17 @@ export const start = async (_zcf, privateArgs, baggage) => { // const { makeRecorderKit } = prepareRecorderKitMakers(baggage, marshaller); const connectionHandler = Far('ConnectionHandler', { - onOpen(connection, localAddr, remoteAddr) { + async onOpen(connection, localAddr, remoteAddr) { // XXX this seems flipped? onOpen(connection, remoteAddr, localAddr) trace('onOpen', JSON.stringify({ localAddr, remoteAddr })); trace('onOpen connection', connection); - return '{"result":"AQ=="}'; + // return '{"result":"AQ=="}'; }, - onClose(_connection, reason) { + async onClose(_connection, reason) { trace('onClose', reason); // XXX what should the behavior be here? }, - onReceive(_connection, bytes) { + async onReceive(_connection, bytes) { /// XXX maybe should throw an error since ica connections will not receive packets? // the types want us to return something here trace('onReceive', bytes);