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

Making EPSS float formatting consistent in 'view Finding' #9591

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

cneill
Copy link
Contributor

@cneill cneill commented Feb 20, 2024

Description

This updates the EPSS score to display 2 decimal places of precision instead of 3 in the "view Finding" view to be consistent with its presentation in the "list Findings" view. Displaying 3 decimal places seems like overkill even if EPSS can go out to many decimal places, since it's technically 5 decimal places with the multiplication by 100 to get a percentage. I just forgot to post a comment about this on the original PR #9516 before it was merged.

Screenshot 2024-02-20 at 12 48 54

Copy link

Contextual Security Analysis

As DryRun Security performs checks, we’ll summarize them here. You can always dive into the detailed results in the section below for checks.

Status DryRun Security Check
Sensitive Functions Analyzer
Configured Sensitive Files Analyzer
Sensitive Files Analyzer

Chat with your AI-powered Security Buddy by typing @dryrunsecurity followed by your question into a comment.
Example: @dryrunsecurity What are common security issues with web application cookies?

Install and configure more repositories at DryRun Security

@github-actions github-actions bot added the ui label Feb 20, 2024
Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

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

Approved

@Maffooch Maffooch merged commit 053929c into DefectDojo:dev Feb 28, 2024
120 of 121 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants