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

Sort missing severity as lowest value in GMP get #1508

Merged

Conversation

timopollmeier
Copy link
Member

@timopollmeier timopollmeier commented Apr 29, 2021

What:
When a filter sorting by severity (or similar fields like cvss_base) is
used in GMP get_... commands, missing values will be sorted as lower
than any other value like 0.0 (log) or -1.0 (false positive).

Why:
When sorting SecInfo items by descending severity, those with known
high severity should be shown first. Items without severity info are often
deprecated, so they should be shown last.

How did you test it:
By sorting various SecInfo list pages by severity in GSA.

Checklist:

When a filter sorting by severity (or similar fields like cvss_base) is
used in GMP get_... commands, missing values will be sorted as lower
than any other value like 0.0 (log) or -1.0 (false positive).
@timopollmeier timopollmeier marked this pull request as ready for review April 29, 2021 07:48
@timopollmeier timopollmeier requested a review from a team as a code owner April 29, 2021 07:48
@timopollmeier timopollmeier added backport-to-stable This pull request will be backported to the stable branch backport-to-main This pull request will be ported to the master branch labels Apr 29, 2021
@nichtsfrei nichtsfrei merged commit 8dce2c6 into greenbone:gvmd-20.08 Apr 29, 2021
timopollmeier added a commit that referenced this pull request May 3, 2021
Sort missing severity as lowest value in GMP get (backport #1508)
timopollmeier added a commit that referenced this pull request May 3, 2021
Sort missing severity as lowest value in GMP get (backport #1508)
@timopollmeier timopollmeier deleted the missing-severity-sorting branch October 15, 2021 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-main This pull request will be ported to the master branch backport-to-stable This pull request will be backported to the stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants