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
+26 -9
View File
@@ -128794,8 +128794,21 @@ class JavaBase {
else {
core.info('Trying to resolve the latest version from remote');
try {
const javaRelease = yield this.findPackageForDownload(this.version);
core.info(`Resolved latest version as ${javaRelease.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}`);
javaRelease.version = jdkPkg.version;
javaRelease.url = jdkPkg.url;
}
if ((foundJava === null || foundJava === void 0 ? void 0 : foundJava.version) === javaRelease.version) {
core.info(`Resolved Java ${foundJava.version} from tool-cache`);
}
@@ -128806,16 +128819,20 @@ class JavaBase {
}
}
catch (error) {
if (error instanceof tc.HTTPError && error.httpStatusCode === 403) {
core.error(`Received HTTP 403: Permission denied or restricted access.`);
}
else if (error instanceof tc.HTTPError &&
error.httpStatusCode === 429) {
core.warning(`Received HTTP 429: Rate limit exceeded. Try again later.`);
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(`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);
+25 -15
View File
@@ -52,8 +52,22 @@ export abstract class JavaBase {
} else {
core.info('Trying to resolve the latest version from remote');
try {
const javaRelease = await this.findPackageForDownload(this.version);
core.info(`Resolved latest version as ${javaRelease.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}`);
javaRelease.version = jdkPkg.version;
javaRelease.url = jdkPkg.url;
}
if (foundJava?.version === javaRelease.version) {
core.info(`Resolved Java ${foundJava.version} from tool-cache`);
} else {
@@ -62,28 +76,24 @@ export abstract class JavaBase {
core.info(`Java ${foundJava.version} was downloaded`);
}
} catch (error: any) {
if (error instanceof tc.HTTPError && error.httpStatusCode === 403) {
core.error(
`Received HTTP 403: Permission denied or restricted access.`
);
} else if (
error instanceof tc.HTTPError &&
error.httpStatusCode === 429
) {
core.warning(
`Received HTTP 429: Rate limit exceeded. Try again later.`
);
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(
`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;
}
}