chore: cleanup console logs

Author: VisargD

src/handlers/handlerUtils.ts

@@ -787,11 +787,16 @@ export async function tryTargetsRecursively(
           );
         }
       } catch (error: any) {
-        console.error('tryTargetsRecursively error: ', error);
         // tryPost always returns a Response.
         // TypeError will check for all unhandled exceptions.
         // GatewayError will check for all handled exceptions which cannot allow the request to proceed.
         if (error instanceof TypeError || error instanceof GatewayError) {
+          console.error(
+            'tryTargetsRecursively error: ',
+            error.message,
+            error.cause,
+            error.stack
+          );
           const errorMessage =
             error instanceof GatewayError
               ? error.message

src/handlers/retryHandler.ts

@@ -175,7 +175,6 @@ export const retryRequest = async (
         retries: retryCount,
         onRetry: (error: Error, attempt: number) => {
           lastAttempt = attempt;
-          console.warn(`Failed in Retry attempt ${attempt}. Error: ${error}`);
         },
         randomize: false,
       }
@@ -186,13 +185,18 @@ export const retryRequest = async (
       error.cause instanceof Error &&
       error.cause?.name === 'ConnectTimeoutError'
     ) {
-      console.error('ConnectTimeoutError: ', error.cause);
+      console.error(
+        'retryRequest ConnectTimeoutError error:',
+        error.cause,
+        error.message
+      );
       // This error comes in case the host address is unreachable. Empty status code used to get returned
       // from here hence no retry logic used to get called.
       lastResponse = new Response(error.message, {
         status: 503,
       });
     } else if (!error.status || error instanceof TypeError) {
+      console.error('retryRequest error:', error.cause, error.message);
       // The retry handler will always attach status code to the error object
       lastResponse = new Response(
         `Message: ${error.message} Cause: ${error.cause ?? 'NA'} Name: ${error.name}`,
@@ -206,9 +210,6 @@ export const retryRequest = async (
         headers: error.headers,
       });
     }
-    console.warn(
-      `Tried ${lastAttempt ?? 1} time(s) but failed. Error: ${JSON.stringify(error)}`
-    );
   }
   return {
     response: lastResponse as Response,

src/index.ts

@@ -109,7 +109,7 @@ app.notFound((c) => c.json({ message: 'Not Found', ok: false }, 404));
  * Otherwise, logs the error and returns a JSON response with status code 500.
  */
 app.onError((err, c) => {
-  console.error('Global Error Handler: ', err);
+  console.error('Global Error Handler: ', err.message, err.cause, err.stack);
   if (err instanceof HTTPException) {
     return err.getResponse();
   }