Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Johnmcg12 sast create report perf #68

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

johnmcg12
Copy link

Peformance fix for SAST-Create-Report (by requesting one report at a time and getting the report before requesting the next one)
Moved credentials to Windows Credentials Manager.
Moved the input for the teams for the reports to ReportTeams.txt.
Created batch files to run the scripts that just take the CxSAST server URL as a parameter.

The report produces a CSV with the standard SAST CSV report format, with the additon of the team path and project name as columns.  The latest scan for each project in the teams provided will be included in this single CSV, each scan results list getting appended to the file.  The teams are supplied by ReportTeams.txt file, one line per team path.  The batch file included shows the example command to run the PowerShell.  To run the batch file supply the SAST URL and direct log output to a file.
SAST scans report - single large CSV output
Moved credentials to Credentials Manager
Moved the teams to a file input: ReportTeams.txt
Performance fix to request each report one at at time so as not to overload the manager with requests.
Created a suggested batch file to run the script in default PDF output mode, that takes the server URL as a parameter.
@nleach999
Copy link
Collaborator

@johnmcg12 please remove the ReportTeams.txt and the .bat files.

Also, it looks like the ability to choose the CSV report type and specify teams was incorporated into the SAST-Create-Report.ps1, and the SAST-Create-CSV-Report.ps1 repeats much of the same logic. It would be better to consolidate everything into SAST-Create-Report.ps1 to allow for report type selection so there is not a need to maintain different scripts.

@nleach999
Copy link
Collaborator

@johnmcg12 please also look at PR #66 by @james-bostock-cx and merge that into the code if it hasn't been done already.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants