fix(connext): set default lookupPayment status as pending (#1603)

* fix(connext): set default lookupPayment status as pending
ExchangeUnion · Jun 5, 2020 · 141bcc5 · 141bcc5
1 parent ebb715e
commit 141bcc5
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/connextclient/ConnextClient.ts b/lib/connextclient/ConnextClient.ts
@@ -401,6 +401,8 @@ class ConnextClient extends SwapClient {
     try {
       const assetId = this.getTokenAddress(currency);
       const transferStatusResponse = await this.getHashLockStatus(rHash, assetId);
+      // TODO: Edge case. Once https://github.com/connext/rest-api-client/issues/31 is
+      // implemented check for the response code 404 and set the payment as failed.
 
       switch (transferStatusResponse.status) {
         case 'PENDING':
@@ -414,10 +416,11 @@ class ConnextClient extends SwapClient {
         case 'FAILED':
           return { state: PaymentState.Failed };
         default:
-          return { state: PaymentState.Failed };
+          return { state: PaymentState.Pending };
       }
     } catch (e) {
-      throw errors.PAYMENT_NOT_FOUND;
+      this.logger.error(e);
+      throw errors.SERVER_ERROR;
     }
   }