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.

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

Meta data summary in JSON shall contain severity totals #2906

Closed
de-jcup opened this issue Feb 16, 2024 · 1 comment · Fixed by #2873
Closed

Meta data summary in JSON shall contain severity totals #2906

de-jcup opened this issue Feb 16, 2024 · 1 comment · Fixed by #2873
Assignees
Milestone

Comments

@de-jcup
Copy link
Member

de-jcup commented Feb 16, 2024

Situation

Currently traffic light totals are inside the json report:

"summary" : {
      "codeScan" : {
        "total" : 5,
        "red" : 0,
        "yellow" : 1,
        "green" : 4,
        "details" : {
          "medium" : [ {
            "name" : "name1",
            "count" : 1
          } ],
          "low" : [ {
            "name" : "name2",
            "count" : 1
          } ],
          "info" : [ {
            "name" : "name3",
            "count" : 2
          },
          {
            "name" : "name4",
            "count" : 1
          },
 ]
        }
      }
    }

Wanted

The scan type summary shall not group by traffic lights, but by severity

"summary" : {
      "codeScan" : {
        "total" : 5,
        "critical" : 0,
        "high" : 0,
        "medium" : 1,
        "low" : 1,
        "unclassified" : 0,
        "info" : 3,
        "details" : {
          "medium" : [ {
            "name" : "name1",
            "count" : 1
          } ],
          "low" : [ {
            "name" : "name2",
            "count" : 1
          } ],
          "info" : [ {
            "name" : "name3",
            "count" : 2
          },{
            "name" : "name4",
            "count" : 1
          }]
        }
      }
    }

Solution

Change implementation

@de-jcup
Copy link
Member Author

de-jcup commented Feb 16, 2024

This is related to

We want to ensure HTML and JSON report have always same data structure.

de-jcup added a commit that referenced this issue Feb 16, 2024
…n and more (#2873)

- Summary in Reports #345
- Fix Email address typos #2874
- Example domain names now only example.com or example.org #2876
- Build artifact sechub-integrationtest-test-reports name issues resolved #2878
- Fixed report issue #2882
- Moved logic from report templates to java classes #2887
- HTML Report summary data grouped now by severity #2893
- Improved TrafficLight /Severity implemenation/handling #2894
- Moved scan report creation logic to own class #2897
- Changes on new HTML report #2898
- Rename main HTML report template #2903
- Removed web development mode and css file #2902
- Added traffic light anker links to HTML report #2904
- Removed unnecessary report summary meta data from json #2889
- Meta data summary in JSON contains severity totals #2906

---------

Co-authored-by: Valentyn Grygoriev <zigfridus@gmail.com>
Co-authored-by: Jeeppler <Jeeppler@users.noreply.github.com>
@de-jcup de-jcup added this to the Server 1.5.0 milestone Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant