fix: ConnectionHandler methods should return promises

Agoric · Mar 25, 2024 · b1184b5 · b1184b5
1 parent 6d2d41b
commit b1184b5
Showing 1 changed file with 4 additions and 4 deletions.
diff --git 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);