Improve Error Handling for Setup-Java Action to Help Debug Intermittent Failures (#848)

* error-handling

* Logic update

---------

Co-authored-by: “gowridurgad” <“hgowridurgad@github.com>

dist/setup/index.js

@@ -128806,16 +128806,20 @@ class JavaBase {
                     }
                 }
                 catch (error) {
-                    if (error instanceof tc.HTTPError && error.httpStatusCode === 403) {
+                    if (error instanceof tc.HTTPError) {
-                        core.error(`Received HTTP 403: Permission denied or restricted access.`);
+                        if (error.httpStatusCode === 403) {
-                    }
+                            core.error('HTTP 403: Permission denied or access restricted.');
-                    else if (error instanceof tc.HTTPError &&
+                        }
-                        error.httpStatusCode === 429) {
+                        else if (error.httpStatusCode === 429) {
-                        core.warning(`Received HTTP 429: Rate limit exceeded. Try again later.`);
+                            core.warning('HTTP 429: Rate limit exceeded. Please retry later.');
+                        }
+                        else {
+                            core.error(`HTTP ${error.httpStatusCode}: ${error.message}`);
+                        }
                     }
                     else {
                         const message = error instanceof Error ? error.message : JSON.stringify(error);
-                        core.error(`Failed to set up Java due to a network issue or timeout: ${message}`);
+                        core.error(`Java setup failed due to network issue or timeout: ${message}`);
                     }
                     if (error instanceof Error && error.stack) {
                         core.debug(error.stack);

src/distributions/base-installer.ts

@@ -62,28 +62,24 @@ export abstract class JavaBase {
           core.info(`Java ${foundJava.version} was downloaded`);
         }
       } catch (error: any) {
-        if (error instanceof tc.HTTPError && error.httpStatusCode === 403) {
+        if (error instanceof tc.HTTPError) {
-          core.error(
+          if (error.httpStatusCode === 403) {
-            `Received HTTP 403: Permission denied or restricted access.`
+            core.error('HTTP 403: Permission denied or access restricted.');
-          );
+          } else if (error.httpStatusCode === 429) {
-        } else if (
+            core.warning('HTTP 429: Rate limit exceeded. Please retry later.');
-          error instanceof tc.HTTPError &&
+          } else {
-          error.httpStatusCode === 429
+            core.error(`HTTP ${error.httpStatusCode}: ${error.message}`);
-        ) {
+          }
-          core.warning(
-            `Received HTTP 429: Rate limit exceeded. Try again later.`
-          );
         } else {
           const message =
             error instanceof Error ? error.message : JSON.stringify(error);
           core.error(
-            `Failed to set up Java due to a network issue or timeout: ${message}`
+            `Java setup failed due to network issue or timeout: ${message}`
           );
         }
         if (error instanceof Error && error.stack) {
           core.debug(error.stack);
         }
-
         throw error;
       }
     }