diff --git a/__tests__/distributors/oracle-installer.test.ts b/__tests__/distributors/oracle-installer.test.ts index 2f133fc9..ad16ecac 100644 --- a/__tests__/distributors/oracle-installer.test.ts +++ b/__tests__/distributors/oracle-installer.test.ts @@ -27,6 +27,11 @@ describe('findPackageForDownload', () => { '20', 'https://download.oracle.com/java/20/latest/jdk-20_{{OS_TYPE}}-x64_bin.{{ARCHIVE_TYPE}}' ], + [ + '18', + '18', + 'https://download.oracle.com/java/18/archive/jdk-18_{{OS_TYPE}}-x64_bin.{{ARCHIVE_TYPE}}' + ], [ '20.0.1', '20.0.1', @@ -43,7 +48,7 @@ describe('findPackageForDownload', () => { 'https://download.oracle.com/java/17/archive/jdk-17.0.1_{{OS_TYPE}}-x64_bin.{{ARCHIVE_TYPE}}' ] ])('version is %s -> %s', async (input, expectedVersion, expectedUrl) => { - /* Needed only for this particular test because /latest/ urls tend to change */ + /* Needed only for this particular test because some urls might change */ spyHttpClient = jest.spyOn(HttpClient.prototype, 'head'); spyHttpClient.mockReturnValue( Promise.resolve({ @@ -53,6 +58,19 @@ describe('findPackageForDownload', () => { }) ); + /** + * NOTE - Should fail to retrieve 18 from latest and check archive instead + */ + if (input === '18') { + spyHttpClient.mockReturnValueOnce( + Promise.resolve({ + message: { + statusCode: 404 + } + }) + ); + } + const result = await distribution['findPackageForDownload'](input); jest.restoreAllMocks(); @@ -75,7 +93,7 @@ describe('findPackageForDownload', () => { jest.spyOn(os, 'arch').mockReturnValue(osArch); jest.spyOn(os, 'platform').mockReturnValue('linux'); - const version = '17'; + const version = '18'; const distro = new OracleDistribution({ version, architecture: '', // to get default value @@ -89,7 +107,7 @@ describe('findPackageForDownload', () => { } const archiveType = getDownloadArchiveExtension(); const result = await distro['findPackageForDownload'](version); - const expectedUrl = `https://download.oracle.com/java/17/latest/jdk-17_${osType}-${distroArch}_bin.${archiveType}`; + const expectedUrl = `https://download.oracle.com/java/18/archive/jdk-18_${osType}-${distroArch}_bin.${archiveType}`; expect(result.url).toBe(expectedUrl); } @@ -102,8 +120,5 @@ describe('findPackageForDownload', () => { await expect(distribution['findPackageForDownload']('11')).rejects.toThrow( /Oracle JDK is only supported for JDK 17 and later/ ); - await expect(distribution['findPackageForDownload']('18')).rejects.toThrow( - /Could not find Oracle JDK for SemVer */ - ); }); }); diff --git a/dist/setup/index.js b/dist/setup/index.js index 2476f9f7..2797eb6c 100644 --- a/dist/setup/index.js +++ b/dist/setup/index.js @@ -103092,27 +103092,33 @@ class OracleDistribution extends base_installer_1.JavaBase { } const platform = this.getPlatform(); const extension = util_1.getDownloadArchiveExtension(); - let major; - let fileUrl; - if (range.includes('.')) { - major = range.split('.')[0]; - fileUrl = `${ORACLE_DL_BASE}/${major}/archive/jdk-${range}_${platform}-${arch}_bin.${extension}`; - } - else { - major = range; - fileUrl = `${ORACLE_DL_BASE}/${range}/latest/jdk-${range}_${platform}-${arch}_bin.${extension}`; + const isOnlyMajorProvided = !range.includes('.'); + const major = isOnlyMajorProvided ? range : range.split('.')[0]; + const possibleUrls = []; + /** + * NOTE + * If only major version was provided we will check it under /latest first + * in order to retrieve the latest possible version if possible, + * otherwise we will fall back to /archive where we are guaranteed to + * find any version if it exists + */ + if (isOnlyMajorProvided) { + possibleUrls.push(`${ORACLE_DL_BASE}/${major}/latest/jdk-${major}_${platform}-${arch}_bin.${extension}`); } + possibleUrls.push(`${ORACLE_DL_BASE}/${major}/archive/jdk-${range}_${platform}-${arch}_bin.${extension}`); if (parseInt(major) < 17) { throw new Error('Oracle JDK is only supported for JDK 17 and later'); } - const response = yield this.http.head(fileUrl); - if (response.message.statusCode === http_client_1.HttpCodes.NotFound) { - throw new Error(`Could not find Oracle JDK for SemVer ${range}`); + for (const url of possibleUrls) { + const response = yield this.http.head(url); + if (response.message.statusCode === http_client_1.HttpCodes.OK) { + return { url, version: range }; + } + if (response.message.statusCode !== http_client_1.HttpCodes.NotFound) { + throw new Error(`Http request for Oracle JDK failed with status code: ${response.message.statusCode}`); + } } - if (response.message.statusCode !== http_client_1.HttpCodes.OK) { - throw new Error(`Http request for Oracle JDK failed with status code: ${response.message.statusCode}`); - } - return { url: fileUrl, version: range }; + throw new Error(`Could not find Oracle JDK for SemVer ${range}`); }); } getPlatform(platform = process.platform) { diff --git a/src/distributions/oracle/installer.ts b/src/distributions/oracle/installer.ts index 2093339a..ec0ab215 100644 --- a/src/distributions/oracle/installer.ts +++ b/src/distributions/oracle/installer.ts @@ -65,33 +65,48 @@ export class OracleDistribution extends JavaBase { const platform = this.getPlatform(); const extension = getDownloadArchiveExtension(); - let major; - let fileUrl; - if (range.includes('.')) { - major = range.split('.')[0]; - fileUrl = `${ORACLE_DL_BASE}/${major}/archive/jdk-${range}_${platform}-${arch}_bin.${extension}`; - } else { - major = range; - fileUrl = `${ORACLE_DL_BASE}/${range}/latest/jdk-${range}_${platform}-${arch}_bin.${extension}`; + + const isOnlyMajorProvided = !range.includes('.'); + const major = isOnlyMajorProvided ? range : range.split('.')[0]; + + const possibleUrls: string[] = []; + + /** + * NOTE + * If only major version was provided we will check it under /latest first + * in order to retrieve the latest possible version if possible, + * otherwise we will fall back to /archive where we are guaranteed to + * find any version if it exists + */ + if (isOnlyMajorProvided) { + possibleUrls.push( + `${ORACLE_DL_BASE}/${major}/latest/jdk-${major}_${platform}-${arch}_bin.${extension}` + ); } + possibleUrls.push( + `${ORACLE_DL_BASE}/${major}/archive/jdk-${range}_${platform}-${arch}_bin.${extension}` + ); + if (parseInt(major) < 17) { throw new Error('Oracle JDK is only supported for JDK 17 and later'); } - const response = await this.http.head(fileUrl); + for (const url of possibleUrls) { + const response = await this.http.head(url); - if (response.message.statusCode === HttpCodes.NotFound) { - throw new Error(`Could not find Oracle JDK for SemVer ${range}`); + if (response.message.statusCode === HttpCodes.OK) { + return {url, version: range}; + } + + if (response.message.statusCode !== HttpCodes.NotFound) { + throw new Error( + `Http request for Oracle JDK failed with status code: ${response.message.statusCode}` + ); + } } - if (response.message.statusCode !== HttpCodes.OK) { - throw new Error( - `Http request for Oracle JDK failed with status code: ${response.message.statusCode}` - ); - } - - return {url: fileUrl, version: range}; + throw new Error(`Could not find Oracle JDK for SemVer ${range}`); } public getPlatform(platform: NodeJS.Platform = process.platform): OsVersions {