Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

feat(cwe): adding CWE results into all reports #6876

Merged
merged 28 commits into from
Feb 21, 2024
Merged

feat(cwe): adding CWE results into all reports #6876

merged 28 commits into from
Feb 21, 2024

Conversation

ArturRibeiro-CX
Copy link
Contributor

Proposed Changes

  • adding CWE results into all report formats

I submit this contribution under the Apache-2.0 license.

@github-actions github-actions bot added the feature New feature label Feb 9, 2024
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 9, 2024

kics-logo

KICS version: v1.7.12

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 1
Files parsed placeholder 1
Files failed to scan placeholder 0
Total executed queries placeholder 49
Queries failed to execute placeholder 0
Execution time placeholder 1

@ArturRibeiro-CX
fix linting pdf and helpers
55cf44e
@ArturRibeiro-CX
linting problem with double critical reference
ff3df23
@ArturRibeiro-CX
fix e2e error and implement html report format
055b876
@ArturRibeiro-CX
remove unused function and fix linting
8d0d319
@ArturRibeiro-CX
changes to csv to accept cwe collumn
57e42bc
@ArturRibeiro-CX
fix string to int error on search line
f541ca3
@ArturRibeiro-CX
cwe addition to csv, result schemas and xml
f2c6846
@ArturRibeiro-CX
change cwe to non required
96f80a8
@ArturRibeiro-CX
unit test for code_climate format
34c7d36
@ArturRibeiro-CX
change helpers for linting problems and added csv unit test
7493ad2
@ArturRibeiro-CX
added unit tests to glsast, sonarqube and junit
107e090
@ArturRibeiro-CX
added unit tests to pdf, json, html and all printers
50e9fd1
@ArturRibeiro-CX
added unit test for asff format, cycloneDX missing
629a0fb
@ArturRibeiro-CX
add last unit test for cycloneDX
a5ae8bc
@ArturRibeiro-CX
change CWE to Cwe to maintain same logic
bdd185d
@ArturRibeiro-CX
change asff back to CWE
18b0ad7
@ArturRibeiro-CX
change back some schemas to CWE
bf7d1e8
@ArturRibeiro-CX
test false additional_Properties csv format
8f05975
@ArturRibeiro-CX
changed cwe to required item on csv format
f77d789
@ArturRibeiro-CX
revert changes on csv
90828a5
cxMiguelSilva
cxMiguelSilva approved these changes Feb 15, 2024
View reviewed changes
Copy link
Collaborator

@cxMiguelSilva cxMiguelSilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@ArturRibeiro-CX ArturRibeiro-CX self-assigned this Feb 16, 2024
@asofsilva asofsilva merged commit 3eba9f6 into master Feb 21, 2024
20 checks passed
@asofsilva asofsilva deleted the cwe_report_formats branch February 21, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cxMiguelSilva cxMiguelSilva cxMiguelSilva approved these changes

Assignees

@ArturRibeiro-CX ArturRibeiro-CX

Labels
feature New feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ArturRibeiro-CX @cxMiguelSilva @asofsilva