fix: windows support requires absolute path to manifest

Signed-off-by: Ilona Shishov <[email protected]>
RHEcosystemAppEng · Jul 15, 2024 · 81041cc · 81041cc
1 parent 7ac4384
commit 81041cc
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 9 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -138077,6 +138077,7 @@ const vulnerabilitySeverityOrder = {
 
 
 
+
 /**
  * Resolves the manifest file path and its corresponding ecosystem.
  * @returns A promise that resolves to an object containing the manifest file path and ecosystem.
@@ -138088,7 +138089,7 @@ async function resolveManifestFilePath() {
     if (!manifestDirInput) {
         core.info(`"${Inputs.MANIFEST_DIRECTORY}" not provided. Using working directory "${process.cwd()}"`);
     }
-    const manifestDir = manifestDirInput || DEFAULT_MANIFEST_DIR;
+    const manifestDir = manifestDirInput ? external_path_default().resolve(manifestDirInput) : process.cwd();
     let manifestFilename;
     if (manifestFileInput) {
         manifestFilename = manifestFileInput;
@@ -138098,10 +138099,11 @@ async function resolveManifestFilePath() {
     }
     else {
         core.info(`"${Inputs.MANIFEST_FILE}" input not provided. Auto-detecting manifest file`);
-        core.info(`🔍 Looking for manifest in "${external_path_default().join(process.cwd(), manifestDir)}"...`);
+        core.info(`🔍 Looking for manifest in "${manifestDir}"...`);
         manifestFilename = await autoDetectManifest(manifestDir);
     }
-    const resolvedManifestPath = external_path_default().join(manifestDir, manifestFilename);
+    core.info(`manifestDir: ${manifestDir}, manifestFilename: ${manifestFilename}`);
+    const resolvedManifestPath = getOS() === 'windows' ? `${manifestDir}\\${manifestFilename}` : `${manifestDir}/${manifestFilename}`;
     core.info(`ℹ️ Manifest file path is "${resolvedManifestPath}"`);
     return {
         manifestFilePath: resolvedManifestPath,
@@ -140934,7 +140936,6 @@ function imageAnalysisService(images, options) {
 async function stackAnalysisService(pathToManifest, options) {
     try {
         // Get stack analysis in JSON format
-        console.log(`pathToManifest: ${pathToManifest}`);
         const stackAnalysisReportJson = await src.stackAnalysis(pathToManifest, false, options);
         return stackAnalysisReportJson;
     }
@@ -141103,6 +141104,7 @@ function getDependencyAnalysisConfig() {
  * @returns A promise that resolves to an object containing the RHDA report JSON and file path.
  */
 async function generateRHDAReport(manifestFilePath, ecosystem) {
+    core.info(`pathToManifest: ${manifestFilePath}`);
     core.info(`⏳ Analysing dependencies...`);
     let rhdaReportJson;
     if (ecosystem === DOCKER) {

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/exhortServices.ts b/src/exhortServices.ts
@@ -59,7 +59,6 @@ async function stackAnalysisService(
 ): Promise<string | exhort.AnalysisReport> {
     try {
         // Get stack analysis in JSON format
-        console.log(`pathToManifest: ${pathToManifest}`)
         const stackAnalysisReportJson = await exhort.stackAnalysis(
             pathToManifest,
             false,

diff --git a/src/manifestHandler.ts b/src/manifestHandler.ts
@@ -7,6 +7,7 @@ import {
     fileNameToEcosystemMappings,
     DEFAULT_MANIFEST_DIR,
 } from './constants.js';
+import { getOS } from './utils.js';
 
 /**
  * Resolves the manifest file path and its corresponding ecosystem.
@@ -25,7 +26,7 @@ export async function resolveManifestFilePath(): Promise<{
             `"${Inputs.MANIFEST_DIRECTORY}" not provided. Using working directory "${process.cwd()}"`,
         );
     }
-    const manifestDir = manifestDirInput || DEFAULT_MANIFEST_DIR;
+    const manifestDir = manifestDirInput ? path.resolve(manifestDirInput) : process.cwd();
 
     let manifestFilename: string;
     if (manifestFileInput) {
@@ -39,13 +40,16 @@ export async function resolveManifestFilePath(): Promise<{
             `"${Inputs.MANIFEST_FILE}" input not provided. Auto-detecting manifest file`,
         );
         ghCore.info(
-            `🔍 Looking for manifest in "${path.join(process.cwd(), manifestDir)}"...`,
+            `🔍 Looking for manifest in "${manifestDir}"...`,
         );
 
         manifestFilename = await autoDetectManifest(manifestDir);
     }
 
-    const resolvedManifestPath = path.join(manifestDir, manifestFilename);
+    ghCore.info(
+        `manifestDir: ${manifestDir}, manifestFilename: ${manifestFilename}`,
+    );
+    const resolvedManifestPath = getOS() === 'windows' ? `${manifestDir}\\${manifestFilename}` : `${manifestDir}/${manifestFilename}`
     ghCore.info(`ℹ️ Manifest file path is "${resolvedManifestPath}"`);
 
     return {

diff --git a/src/rhda.ts b/src/rhda.ts
@@ -45,6 +45,7 @@ export async function generateRHDAReport(
     manifestFilePath: string,
     ecosystem: string,
 ): Promise<{ rhdaReportJson: any; rhdaReportJsonFilePath: string }> {
+    ghCore.info(`pathToManifest: ${manifestFilePath}`);
     ghCore.info(`⏳ Analysing dependencies...`);
 
     let rhdaReportJson: string | any;