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

Check if vulnerability severity exists when calculating vulnerability score #80

Merged
merged 3 commits into from
Sep 5, 2023
Merged

Check if vulnerability severity exists when calculating vulnerability score #80

merged 3 commits into from
Sep 5, 2023

Conversation

tamasmak
Copy link
Contributor

@tamasmak tamasmak commented Sep 4, 2023

Fixes #79

@tamasmak tamasmak self-assigned this Sep 4, 2023
@tamasmak tamasmak requested a review from heruan September 4, 2023 21:29
heruan
heruan requested changes Sep 5, 2023
View reviewed changes
pom.xml Outdated Show resolved Hide resolved
heruan
heruan approved these changes Sep 5, 2023
View reviewed changes
Copy link
Member

@heruan heruan left a comment

Choose a reason for hiding this comment

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

We should test this along with other risky nullable attributes from external sources. I've create #83 to track this.

@heruan heruan merged commit 22e5414 into main Sep 5, 2023
@heruan heruan deleted the fix/severity-null-check branch September 5, 2023 09:16
@tamasmak tamasmak mentioned this pull request Sep 27, 2023
Closed
heruan pushed a commit that referenced this pull request Oct 12, 2023
@tamasmak @heruan
fix: Check if vulnerability severity exists when calculating vulnerab…
abb59f8 
…ility score (#80)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@heruan heruan heruan approved these changes

Assignees

@tamasmak tamasmak

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

NPE <= OpenSourceVulnerability.Severity
2 participants
@tamasmak @heruan