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

* error-handling

* Logic update

---------

Co-authored-by: “gowridurgad” <“[email protected]>

Author: gowridurgad

dist/setup/index.js

@@ -128793,15 +128793,38 @@ class JavaBase {
             }
             else {
                 core.info('Trying to resolve the latest version from remote');
-                const javaRelease = yield this.findPackageForDownload(this.version);
-                core.info(`Resolved latest version as ${javaRelease.version}`);
-                if ((foundJava === null || foundJava === void 0 ? void 0 : foundJava.version) === javaRelease.version) {
-                    core.info(`Resolved Java ${foundJava.version} from tool-cache`);
+                try {
+                    const javaRelease = yield this.findPackageForDownload(this.version);
+                    core.info(`Resolved latest version as ${javaRelease.version}`);
+                    if ((foundJava === null || foundJava === void 0 ? void 0 : foundJava.version) === javaRelease.version) {
+                        core.info(`Resolved Java ${foundJava.version} from tool-cache`);
+                    }
+                    else {
+                        core.info('Trying to download...');
+                        foundJava = yield this.downloadTool(javaRelease);
+                        core.info(`Java ${foundJava.version} was downloaded`);
+                    }
                 }
-                else {
-                    core.info('Trying to download...');
-                    foundJava = yield this.downloadTool(javaRelease);
-                    core.info(`Java ${foundJava.version} was downloaded`);
+                catch (error) {
+                    if (error instanceof tc.HTTPError) {
+                        if (error.httpStatusCode === 403) {
+                            core.error('HTTP 403: Permission denied or access restricted.');
+                        }
+                        else if (error.httpStatusCode === 429) {
+                            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(`Java setup failed due to network issue or timeout: ${message}`);
+                    }
+                    if (error instanceof Error && error.stack) {
+                        core.debug(error.stack);
+                    }
+                    throw error;
                 }
             }
             // JDK folder may contain postfix "Contents/Home" on macOS

src/distributions/base-installer.ts

@@ -51,14 +51,36 @@ export abstract class JavaBase {
       core.info(`Resolved Java ${foundJava.version} from tool-cache`);
     } else {
       core.info('Trying to resolve the latest version from remote');
-      const javaRelease = await this.findPackageForDownload(this.version);
-      core.info(`Resolved latest version as ${javaRelease.version}`);
-      if (foundJava?.version === javaRelease.version) {
-        core.info(`Resolved Java ${foundJava.version} from tool-cache`);
-      } else {
-        core.info('Trying to download...');
-        foundJava = await this.downloadTool(javaRelease);
-        core.info(`Java ${foundJava.version} was downloaded`);
+      try {
+        const javaRelease = await this.findPackageForDownload(this.version);
+        core.info(`Resolved latest version as ${javaRelease.version}`);
+        if (foundJava?.version === javaRelease.version) {
+          core.info(`Resolved Java ${foundJava.version} from tool-cache`);
+        } else {
+          core.info('Trying to download...');
+          foundJava = await this.downloadTool(javaRelease);
+          core.info(`Java ${foundJava.version} was downloaded`);
+        }
+      } catch (error: any) {
+        if (error instanceof tc.HTTPError) {
+          if (error.httpStatusCode === 403) {
+            core.error('HTTP 403: Permission denied or access restricted.');
+          } else if (error.httpStatusCode === 429) {
+            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(
+            `Java setup failed due to network issue or timeout: ${message}`
+          );
+        }
+        if (error instanceof Error && error.stack) {
+          core.debug(error.stack);
+        }
+        throw error;
       }
     }