Merge pull request #2 from thereisnotime/patch-3

Patch: Set proper default for options
mbogh · Aug 24, 2022 · afe2e89 · afe2e89
2 parents 4e4ac62 + 567a95c
commit afe2e89
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -8,7 +8,7 @@ A GitHub Action for scanning a host with [testssl.sh](https://testssl.sh) and co
 - `image`: Docker image to run testssl.sh (Default: `drwetter/testssl.sh`)
 - `output`: Folder for scan reports (Default: `output`)
 - `grade`: Minimum accepted grade (Default: `A+`)
-- `options`: Additionnal testssl.sh CLI options
+- `options`: Additionnal testssl.sh CLI options (Default: `--jsonfile /data --csvfile /data --htmlfile /data`)
 
 ## Outputs
 
@@ -26,4 +26,4 @@ The scan result will be placed in the output folder, the result will be availabl
   with:
     name: testssl.sh reports
     path: 'output/*'
-```
+```
diff --git a/action.yml b/action.yml
@@ -19,7 +19,7 @@ inputs:
     default: 'A+'
   options:
     description: 'Additionnal testssl.sh CLI options'
-    default: ''
+    default: '--jsonfile /data --csvfile /data --htmlfile /data'
 runs:
   using: 'node12'
-  main: 'dist/index.js'
+  main: 'dist/index.js'
diff --git a/dist/index.js b/dist/index.js
diff --git a/index.js b/index.js
@@ -22,7 +22,7 @@ async function run() {
 
     await exec.exec(`mkdir -p ${workspace}/${output}`);
     await exec.exec(`docker pull ${image} -q`);
-    let command = (`docker run --user 0:0 -v ${workspace}/${output}:/data --network="host" ` + `-t ${image} --jsonfile /data --csvfile /data --htmlfile /data ${options} ${host}`);
+    let command = (`docker run --user 0:0 -v ${workspace}/${output}:/data --network="host" ` + `-t ${image} ${options} ${host}`);
     try {
       await exec.exec(command);