Skip to content

Fix for pdf report not using default engines#587

Merged
cx-pedro-lopes merged 4 commits intomainfrom
BUG/AST-30015-PDF-REPORT-FIX-ENGINES
Sep 29, 2023
Merged

Fix for pdf report not using default engines#587
cx-pedro-lopes merged 4 commits intomainfrom
BUG/AST-30015-PDF-REPORT-FIX-ENGINES

Conversation

@igorlombacx
Copy link
Contributor

@igorlombacx igorlombacx commented Sep 27, 2023

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Fix to always generate a pdf report using engines gotten on the scan if "report-pdf-options" flag value is default

image

References

https://checkmarx.atlassian.net/browse/AST-30015?atlOrigin=eyJpIjoiZTgzODI3M2VmZGNkNDgxNDgxMzlmOGIwM2IyMDRiNTAiLCJwIjoiaiJ9

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • I have updated the CLI help for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@cx-pedro-lopes cx-pedro-lopes requested review from a team, cx-hugo-machado and cx-margarita-levitm and removed request for a team September 27, 2023 15:31
@github-actions github-actions bot added the bug Something isn't working label Sep 27, 2023
@github-actions
Copy link

github-actions bot commented Sep 27, 2023

Logo
Checkmarx One – Scan Summary & Detailsc7fe7a00-7fdf-46ce-9834-88349667a993

No New Or Fixed Issues Found

@cx-pedro-lopes cx-pedro-lopes self-requested a review September 29, 2023 09:07
@cx-pedro-lopes cx-pedro-lopes merged commit 6790ba2 into main Sep 29, 2023
@cx-pedro-lopes cx-pedro-lopes deleted the BUG/AST-30015-PDF-REPORT-FIX-ENGINES branch September 29, 2023 09:09
igorlombacx added a commit that referenced this pull request Oct 20, 2023
* fix for pdf report not using default engines

* fix for pdf report not using default engines

* Update one-scan.yml

* Update one-scan.yml

---------

Co-authored-by: Pedro Lopes <83576881+pedrompflopes@users.noreply.github.com>
cx-ittai-gilat pushed a commit that referenced this pull request Jan 4, 2024
* fix for pdf report not using default engines

* fix for pdf report not using default engines

* Update one-scan.yml

* Update one-scan.yml

---------

Co-authored-by: Pedro Lopes <83576881+pedrompflopes@users.noreply.github.com>
thtri pushed a commit to thtri/ast-cli that referenced this pull request Apr 19, 2024
* fix for pdf report not using default engines

* fix for pdf report not using default engines

* Update one-scan.yml

* Update one-scan.yml

---------

Co-authored-by: Pedro Lopes <83576881+pedrompflopes@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants