-
Notifications
You must be signed in to change notification settings - Fork 365
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
Remove CVE, CVSS, CVRF Features #2108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great!
@@ -675,8 +675,7 @@ int oscap_source_validate_schematron_priv(struct oscap_source *source, oscap_doc | |||
scap_type == OSCAP_DOCUMENT_CPE_DICTIONARY || | |||
scap_type == OSCAP_DOCUMENT_CVE_FEED || | |||
scap_type == OSCAP_DOCUMENT_SCE_RESULT || | |||
scap_type == OSCAP_DOCUMENT_OCIL || | |||
scap_type == OSCAP_DOCUMENT_CVRF_FEED) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to remove remaining occurrences of OSCAP_DOCUMENT_CVRF_FEED from other files in src as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a great removal of obsolete features!
The CI fail on Fedora Latest isn't caused by this PR and will be fixed in #2118.
I have built and run oscap
, reviewed the CLI and I have read the man page.
At this moment, it isn't possible to build and run tests on CentOS Stream 8 because CentOS Stream 8 has been sunset. |
This PR removes CVE, CVSS, and CVRF features.
This is in one PR as the three features are closely related and it was easier to just remove them all at once.
Fixes #2087
Fixes #2088
Fixes #2089