feat: new error handling

package.json

@@ -29,6 +29,7 @@
     "@govtechsg/token-registry": "^2.0.0",
     "debug": "^4.1.1",
     "ethers": "^5.0.4",
+    "msw": "^0.20.5",
     "snyk": "^1.364.2"
   },
   "devDependencies": {

src/types/error.ts

@@ -7,7 +7,9 @@ export enum OpenAttestationEthereumDocumentStoreStatusCode {
   ETHERS_UNHANDLED_ERROR = 3,
   SKIPPED = 4,
   DOCUMENT_REVOKED = 5,
+  INVALID_ARGUMENT = 6,
   CONTRACT_NOT_FOUND = 404,
+  SERVER_ERROR = 500,
 }
 export enum OpenAttestationDocumentSignedCode {
   UNEXPECTED_ERROR = 0,
@@ -21,7 +23,9 @@ export enum OpenAttestationEthereumTokenRegistryStatusCode {
   CONTRACT_ADDRESS_INVALID = 2,
   ETHERS_UNHANDLED_ERROR = 3,
   SKIPPED = 4,
+  INVALID_ARGUMENT = 6,
   CONTRACT_NOT_FOUND = 404,
+  SERVER_ERROR = 500,
 }
 export enum OpenAttestationDnsTxtCode {
   UNEXPECTED_ERROR = 0,

src/common/smartContract/documentStoreErrors.ts → src/verifiers/documentStoreStatus/errors.ts

@@ -9,6 +9,7 @@ const contractNotFound = (address: Hash): Reason => {
     message: `Contract ${address} was not found`,
   };
 };
+
 const contractAddressInvalid = (address: Hash): Reason => {
   return {
     code: OpenAttestationEthereumDocumentStoreStatusCode.CONTRACT_ADDRESS_INVALID,
@@ -19,6 +20,7 @@ const contractAddressInvalid = (address: Hash): Reason => {
     message: `Contract address ${address} is invalid`,
   };
 };
+
 export const contractNotIssued = (merkleRoot: Hash, address: string): Reason => {
   return {
     code: OpenAttestationEthereumDocumentStoreStatusCode.DOCUMENT_NOT_ISSUED,
@@ -39,6 +41,27 @@ export const contractRevoked = (merkleRoot: string, address: string): Reason =>
   };
 };
 
+// This function handles ALL of Ethers SERVER_ERRORs, most likely caused by HTTP 4xx or 5xx errors.
+export const serverError = (): Reason => {
+  return {
+    code: OpenAttestationEthereumDocumentStoreStatusCode.SERVER_ERROR,
+    codeString:
+      OpenAttestationEthereumDocumentStoreStatusCode[OpenAttestationEthereumDocumentStoreStatusCode.SERVER_ERROR],
+    message: `Unable to connect to the Ethereum network, please try again later`,
+  };
+};
+
+// This function handles all INVALID_ARGUMENT errors likely due to invalid hex string,
+// hex data is odd-length or incorrect data length
+export const invalidArgument = (error: EthersError, address: string): Reason => {
+  return {
+    code: OpenAttestationEthereumDocumentStoreStatusCode.INVALID_ARGUMENT,
+    codeString:
+      OpenAttestationEthereumDocumentStoreStatusCode[OpenAttestationEthereumDocumentStoreStatusCode.INVALID_ARGUMENT],
+    message: `Error with smart contract ${address}: ${error.reason}`,
+  };
+};
+
 export const getErrorReason = (error: EthersError, address: string): Reason | null => {
   const reason = error.reason && Array.isArray(error.reason) ? error.reason[0] : error.reason ?? "";
   if (
@@ -55,7 +78,12 @@ export const getErrorReason = (error: EthersError, address: string): Reason | nu
     (reason.toLowerCase() === "invalid address".toLowerCase() && error.code === errors.INVALID_ARGUMENT)
   ) {
     return contractAddressInvalid(address);
+  } else if (error.code === errors.SERVER_ERROR) {
+    return serverError();
+  } else if (error.code === errors.INVALID_ARGUMENT) {
+    return invalidArgument(error, address);
   }
+
   return {
     message: `Error with smart contract ${address}: ${error.reason}`,
     code: OpenAttestationEthereumDocumentStoreStatusCode.ETHERS_UNHANDLED_ERROR,

src/verifiers/documentStoreStatus/openAttestationEthereumDocumentStoreStatus.ts

@@ -3,7 +3,7 @@ import { DocumentStoreFactory } from "@govtechsg/document-store";
 import { DocumentStore } from "@govtechsg/document-store/src/contracts/DocumentStore";
 import { Hash, VerificationFragmentType, VerificationFragment, Verifier } from "../../types/core";
 import { OpenAttestationEthereumDocumentStoreStatusCode } from "../../types/error";
-import { contractNotIssued, getErrorReason, contractRevoked } from "../../common/smartContract/documentStoreErrors";
+import { contractNotIssued, getErrorReason, contractRevoked } from "./errors";
 import { getIssuersDocumentStore, getProvider } from "../../common/utils";
 
 interface IssuanceStatus {

src/verifiers/tokenRegistryStatus/errors.ts

@@ -10,6 +10,7 @@ const contractNotFound = (address: Hash): Reason => {
     message: `Contract ${address} was not found`,
   };
 };
+
 const contractAddressInvalid = (address: Hash): Reason => {
   return {
     code: OpenAttestationEthereumTokenRegistryStatusCode.CONTRACT_ADDRESS_INVALID,
@@ -20,6 +21,7 @@ const contractAddressInvalid = (address: Hash): Reason => {
     message: `Contract address ${address} is invalid`,
   };
 };
+
 export const contractNotMinted = (merkleRoot: Hash, address: string): Reason => {
   return {
     code: OpenAttestationEthereumTokenRegistryStatusCode.DOCUMENT_NOT_MINTED,
@@ -31,6 +33,27 @@ export const contractNotMinted = (merkleRoot: Hash, address: string): Reason =>
   };
 };
 
+// This function handles ALL of Ethers SERVER_ERRORs, most likely caused by HTTP 4xx or 5xx errors.
+export const serverError = (): Reason => {
+  return {
+    code: OpenAttestationEthereumTokenRegistryStatusCode.SERVER_ERROR,
+    codeString:
+      OpenAttestationEthereumTokenRegistryStatusCode[OpenAttestationEthereumTokenRegistryStatusCode.SERVER_ERROR],
+    message: `Unable to connect to the Ethereum network, please try again later`,
+  };
+};
+
+// This function handles all INVALID_ARGUMENT errors likely due to invalid hex string,
+// hex data is odd-length or incorrect data length
+export const invalidArgument = (error: EthersError, address: string): Reason => {
+  return {
+    code: OpenAttestationEthereumTokenRegistryStatusCode.INVALID_ARGUMENT,
+    codeString:
+      OpenAttestationEthereumTokenRegistryStatusCode[OpenAttestationEthereumTokenRegistryStatusCode.INVALID_ARGUMENT],
+    message: `Error with smart contract ${address}: ${error.reason}`,
+  };
+};
+
 export const getErrorReason = (error: EthersError, address: string, hash: Hash): Reason => {
   const reason = error.reason && Array.isArray(error.reason) ? error.reason[0] : error.reason ?? "";
   if (
@@ -50,7 +73,12 @@ export const getErrorReason = (error: EthersError, address: string, hash: Hash):
     (reason.toLowerCase() === "invalid address".toLowerCase() && error.code === errors.INVALID_ARGUMENT)
   ) {
     return contractAddressInvalid(address);
+  } else if (error.code === errors.SERVER_ERROR) {
+    return serverError();
+  } else if (error.code === errors.INVALID_ARGUMENT) {
+    return invalidArgument(error, address);
   }
+
   return {
     message: `Error with smart contract ${address}: ${error.reason}`,
     code: OpenAttestationEthereumTokenRegistryStatusCode.ETHERS_UNHANDLED_ERROR,