Fix: prevent default installation of JetBrains pre-releases (#859)

* fix: prevent default installation of JetBrains pre-releases

* simplify prerelease filter logic

* fix basic validation checks

.github/workflows/publish-immutable-actions.yml

@@ -2,7 +2,8 @@ name: 'Publish Immutable Action Version'
 
 on:
   release:
-    types: [published]
+    types: [released]
+  workflow_dispatch:
 
 jobs:
   publish:

__tests__/distributors/jetbrains-installer.test.ts

@@ -41,9 +41,7 @@ describe('getAvailableVersions', () => {
     expect(availableVersions).not.toBeNull();
 
     const length =
-      os.platform() === 'win32'
+      os.platform() === 'win32' ? manifestData.length : manifestData.length + 2;
-        ? manifestData.length - 1
-        : manifestData.length + 1;
     expect(availableVersions.length).toBe(length);
   }, 10_000);
 });

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);
+                try {
-                core.info(`Resolved latest version as ${javaRelease.version}`);
+                    const javaRelease = yield this.findPackageForDownload(this.version);
-                if ((foundJava === null || foundJava === void 0 ? void 0 : foundJava.version) === javaRelease.version) {
+                    core.info(`Resolved latest version as ${javaRelease.version}`);
-                    core.info(`Resolved Java ${foundJava.version} from tool-cache`);
+                    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 {
+                catch (error) {
-                    core.info('Trying to download...');
+                    if (error instanceof tc.HTTPError) {
-                    foundJava = yield this.downloadTool(javaRelease);
+                        if (error.httpStatusCode === 403) {
-                    core.info(`Java ${foundJava.version} was downloaded`);
+                            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
@@ -129672,7 +129695,12 @@ class JetBrainsDistribution extends base_installer_1.JavaBase {
                     // url is identical except page_index so print it once for debug
                     core.debug(`Gathering available versions from '${rawUrl}'`);
                 }
-                const paginationPage = (yield this.http.getJson(rawUrl, requestHeaders)).result;
+                const paginationPageResult = (yield this.http.getJson(rawUrl, requestHeaders)).result;
+                if (!paginationPageResult || paginationPageResult.length === 0) {
+                    // break infinity loop because we have reached end of pagination
+                    break;
+                }
+                const paginationPage = paginationPageResult.filter(version => this.stable ? !version.prerelease : version.prerelease);
                 if (!paginationPage || paginationPage.length === 0) {
                     // break infinity loop because we have reached end of pagination
                     break;
@@ -129680,9 +129708,11 @@ class JetBrainsDistribution extends base_installer_1.JavaBase {
                 rawVersions.push(...paginationPage);
                 page_index++;
             }
-            // Add versions not available from the API but are downloadable
+            if (this.stable) {
-            const hidden = ['11_0_10b1145.115', '11_0_11b1341.60'];
+                // Add versions not available from the API but are downloadable
-            rawVersions.push(...hidden.map(tag => ({ tag_name: tag, name: tag })));
+                const hidden = ['11_0_10b1145.115', '11_0_11b1341.60'];
+                rawVersions.push(...hidden.map(tag => ({ tag_name: tag, name: tag, prerelease: false })));
+            }
             const versions0 = rawVersions.map((v) => __awaiter(this, void 0, void 0, function* () {
                 var _a;
                 // Release tags look like one of these:
@@ -129701,7 +129731,7 @@ class JetBrainsDistribution extends base_installer_1.JavaBase {
                     .replace('-', '');
                 const vsplit = vstring.split('b');
                 let semver = vsplit[0];
-                const build = +vsplit[1];
+                const build = vsplit[1];
                 // Normalize semver
                 if (!semver.includes('.') && !semver.includes('_'))
                     semver = `${semver}.0.0`;

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);
+      try {
-      core.info(`Resolved latest version as ${javaRelease.version}`);
+        const javaRelease = await this.findPackageForDownload(this.version);
-      if (foundJava?.version === javaRelease.version) {
+        core.info(`Resolved latest version as ${javaRelease.version}`);
-        core.info(`Resolved Java ${foundJava.version} from tool-cache`);
+        if (foundJava?.version === javaRelease.version) {
-      } else {
+          core.info(`Resolved Java ${foundJava.version} from tool-cache`);
-        core.info('Trying to download...');
+        } else {
-        foundJava = await this.downloadTool(javaRelease);
+          core.info('Trying to download...');
-        core.info(`Java ${foundJava.version} was downloaded`);
+          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;
       }
     }
 

src/distributions/jetbrains/installer.ts

@@ -113,9 +113,18 @@ export class JetBrainsDistribution extends JavaBase {
         core.debug(`Gathering available versions from '${rawUrl}'`);
       }
 
-      const paginationPage = (
+      const paginationPageResult = (
         await this.http.getJson<IJetBrainsRawVersion[]>(rawUrl, requestHeaders)
       ).result;
+      if (!paginationPageResult || paginationPageResult.length === 0) {
+        // break infinity loop because we have reached end of pagination
+        break;
+      }
+
+      const paginationPage: IJetBrainsRawVersion[] =
+        paginationPageResult.filter(version =>
+          this.stable ? !version.prerelease : version.prerelease
+        );
       if (!paginationPage || paginationPage.length === 0) {
         // break infinity loop because we have reached end of pagination
         break;
@@ -125,9 +134,13 @@ export class JetBrainsDistribution extends JavaBase {
       page_index++;
     }
 
-    // Add versions not available from the API but are downloadable
+    if (this.stable) {
-    const hidden = ['11_0_10b1145.115', '11_0_11b1341.60'];
+      // Add versions not available from the API but are downloadable
-    rawVersions.push(...hidden.map(tag => ({tag_name: tag, name: tag})));
+      const hidden = ['11_0_10b1145.115', '11_0_11b1341.60'];
+      rawVersions.push(
+        ...hidden.map(tag => ({tag_name: tag, name: tag, prerelease: false}))
+      );
+    }
 
     const versions0 = rawVersions.map(async v => {
       // Release tags look like one of these:
@@ -148,7 +161,7 @@ export class JetBrainsDistribution extends JavaBase {
 
       const vsplit = vstring.split('b');
       let semver = vsplit[0];
-      const build = +vsplit[1];
+      const build = vsplit[1];
 
       // Normalize semver
       if (!semver.includes('.') && !semver.includes('_'))

src/distributions/jetbrains/models.ts

@@ -3,11 +3,12 @@
 export interface IJetBrainsRawVersion {
   tag_name: string;
   name: string;
+  prerelease: boolean;
 }
 
 export interface IJetBrainsVersion {
   tag_name: string;
   semver: string;
-  build: number;
+  build: string;
   url: string;
 }