Merge pull request #667 from Merit-Systems/br/specific-facilitator-failure-message

specific facilitators failed error

Author: zdql

packages/app/server/src/errors/http.ts

@@ -21,14 +21,16 @@ export class PaymentRequiredError extends HttpError {
   }
 }
 
-class MaxInFlightRequestsError extends HttpError {
-  constructor(message: string = 'Max In Flight Requests') {
-    super(429, message);
-  }
-}
-
 export class UnknownModelError extends HttpError {
   constructor(message: string = 'Unknown Model argument passed in') {
     super(400, message);
   }
 }
+
+export class FacilitatorProxyError extends HttpError {
+  constructor(
+    message: string = 'FacilitatorProxyError - all facilitators are busy. Please retry after a few seconds.'
+  ) {
+    super(513, message);
+  }
+}

packages/app/server/src/services/facilitator/facilitatorProxy.ts

@@ -8,6 +8,7 @@ import {
 import logger, { logMetric } from '../../logger';
 import dotenv from 'dotenv';
 import { env } from '../../env';
+import { FacilitatorProxyError } from '../../errors/http';
 
 dotenv.config();
 
@@ -69,13 +70,11 @@ export async function facilitatorProxy<
   clearTimeout(timeoutId);
 
   if (res.status !== 200) {
-    const errorBody = await res.text();
-    const errorMsg = `${res.status} ${res.statusText} - ${errorBody}`;
     logMetric('facilitator_proxy_failure', 1, {
       method,
       status: res.status,
     });
-    throw new Error(`Proxy facilitator failed for ${method}: ${errorMsg}`);
+    throw new FacilitatorProxyError();
   }
 
   const data = await res.json();