Compare commits

..

2 Commits

Author SHA1 Message Date
suisrc d5ddf8cf5e add custom jdk url 2025-06-26 09:05:01 +08:00
gowridurgad ebb356cc4e Improve Error Handling for Setup-Java Action to Help Debug Intermittent Failures (#848)
* error-handling

* Logic update

---------

Co-authored-by: “gowridurgad” <“hgowridurgad@github.com>
2025-06-23 12:32:03 -05:00
2 changed files with 51 additions and 24 deletions
+24 -7
View File
@@ -128794,8 +128794,21 @@ class JavaBase {
else { else {
core.info('Trying to resolve the latest version from remote'); core.info('Trying to resolve the latest version from remote');
try { try {
const javaRelease = yield this.findPackageForDownload(this.version); const javaRelease = { version: this.version };
const jdkUrl = core.getInput('jdk-url');
if (jdkUrl) {
if (!this.version) {
throw new Error('You must specify a version when using a custom JDK URL.');
}
core.info(`Using custom JDK URL: ${jdkUrl}`);
javaRelease.url = jdkUrl;
}
else {
const jdkPkg = yield this.findPackageForDownload(this.version);
core.info(`Resolved latest version as ${javaRelease.version}`); core.info(`Resolved latest version as ${javaRelease.version}`);
javaRelease.version = jdkPkg.version;
javaRelease.url = jdkPkg.url;
}
if ((foundJava === null || foundJava === void 0 ? void 0 : foundJava.version) === javaRelease.version) { if ((foundJava === null || foundJava === void 0 ? void 0 : foundJava.version) === javaRelease.version) {
core.info(`Resolved Java ${foundJava.version} from tool-cache`); core.info(`Resolved Java ${foundJava.version} from tool-cache`);
} }
@@ -128806,16 +128819,20 @@ class JavaBase {
} }
} }
catch (error) { 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.httpStatusCode === 429) {
core.warning('HTTP 429: Rate limit exceeded. Please retry later.');
}
else {
core.error(`HTTP ${error.httpStatusCode}: ${error.message}`);
} }
else if (error instanceof tc.HTTPError &&
error.httpStatusCode === 429) {
core.warning(`Received HTTP 429: Rate limit exceeded. Try again later.`);
} }
else { else {
const message = error instanceof Error ? error.message : JSON.stringify(error); 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) { if (error instanceof Error && error.stack) {
core.debug(error.stack); core.debug(error.stack);
+24 -14
View File
@@ -52,8 +52,22 @@ export abstract class JavaBase {
} else { } else {
core.info('Trying to resolve the latest version from remote'); core.info('Trying to resolve the latest version from remote');
try { try {
const javaRelease = await this.findPackageForDownload(this.version); const javaRelease = { version: this.version } as JavaDownloadRelease;
const jdkUrl = core.getInput('jdk-url');
if (jdkUrl) {
if (!this.version) {
throw new Error(
'You must specify a version when using a custom JDK URL.'
);
}
core.info(`Using custom JDK URL: ${jdkUrl}`);
javaRelease.url = jdkUrl;
} else {
const jdkPkg = await this.findPackageForDownload(this.version)
core.info(`Resolved latest version as ${javaRelease.version}`); core.info(`Resolved latest version as ${javaRelease.version}`);
javaRelease.version = jdkPkg.version;
javaRelease.url = jdkPkg.url;
}
if (foundJava?.version === javaRelease.version) { if (foundJava?.version === javaRelease.version) {
core.info(`Resolved Java ${foundJava.version} from tool-cache`); core.info(`Resolved Java ${foundJava.version} from tool-cache`);
} else { } else {
@@ -62,28 +76,24 @@ export abstract class JavaBase {
core.info(`Java ${foundJava.version} was downloaded`); core.info(`Java ${foundJava.version} was downloaded`);
} }
} catch (error: any) { } 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 { } else {
const message = const message =
error instanceof Error ? error.message : JSON.stringify(error); error instanceof Error ? error.message : JSON.stringify(error);
core.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) { if (error instanceof Error && error.stack) {
core.debug(error.stack); core.debug(error.stack);
} }
throw error; throw error;
} }
} }