allowIssueWriting: Also move to options object

Fixes the same issue of avoiding confusion with parameters are added to the signature. Signed-off-by: Tony Narlock <[email protected]>
zaproxy · Dec 3, 2021 · 78d624c · 78d624c
1 parent b578e13
commit 78d624c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/packages/scans/src/constants.js b/packages/scans/src/constants.js
@@ -1,5 +1,6 @@
 module.exports = {
     DEFAULT_OPTIONS: {
-        artifactName: 'zap_scan'
+      allowIssueWriting: true,
+      artifactName: 'zap_scan',
     }
 }
diff --git a/packages/scans/src/index.js b/packages/scans/src/index.js
@@ -7,7 +7,7 @@ const actionHelper = require('./action-helper');
 const { DEFAULT_OPTIONS } = require('./constants');
 
 let actionCommon = {
-    processReport: (async (token, workSpace, plugins, currentRunnerID, issueTitle, repoName, allowIssueWriting = true, { artifactName } = DEFAULT_OPTIONS) => {
+    processReport: (async (token, workSpace, plugins, currentRunnerID, issueTitle, repoName, { allowIssueWriting, artifactName } = DEFAULT_OPTIONS) => {
         let jsonReportName = 'report_json.json';
         let mdReportName = 'report_md.md';
         let htmlReportName = 'report_html.html';