Merge pull request #246 from metaDAOproject/fix/uncaught-tx-serialize…

…r-error fix/uncaught tx serializer error
metaDAOproject · Aug 19, 2024 · 2efd160 · 2efd160
2 parents bf4669c + 7eaf1fe
commit 2efd160
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 7 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -32,7 +32,7 @@ jobs:
           context: .
           file: packages/indexer/Dockerfile
           push: true
-          tags: ${{ steps.meta.outputs.tags }}
+          tags: themetadao/indexer:main
           labels: ${{ steps.meta.outputs.labels }}
 
   redeploy_app:

diff --git a/packages/indexer/src/transaction/serializer.ts b/packages/indexer/src/transaction/serializer.ts
@@ -18,6 +18,7 @@ import {
   InstructionDisplay,
   Instruction as AnchorInstruction,
 } from "@coral-xyz/anchor/dist/cjs/coder/borsh/instruction";
+import { logger } from "../logger";
 
 /**
  * This version should be bumped every time we update this file.
@@ -401,13 +402,17 @@ function getIxWithDisplay(
       };
     }
   });
+  try {
+    const ixDisplay = coder.format(decodedIx, accounts);
 
-  const ixDisplay = coder.format(decodedIx, accounts);
-
-  return {
-    instruction: decodedIx,
-    instructionDisplay: ixDisplay,
-  };
+    return {
+      instruction: decodedIx,
+      instructionDisplay: ixDisplay,
+    };
+  } catch (e) {
+    logger.error("error with coder formatting of decodedIx:", e);
+    return null;
+  }
 }
 
 function flattenIdlAccounts(