Skip to content

CVE api changes #183

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

Merged
merged 6 commits into from
May 28, 2025
Merged

CVE api changes #183

merged 6 commits into from
May 28, 2025

Conversation

daniele-moro
Copy link
Contributor

@daniele-moro daniele-moro commented May 27, 2025

Description

This pull request introduces changes to enhance the representation of CVE (Common Vulnerabilities and Exposures) data for instances and operating systems in the API. The updates include new fields in the OpenAPI schema, protobuf definitions, and documentation to support the storage and retrieval of CVE-related information.

Additions to CVE Data Representation:

OpenAPI Schema Updates:

  • Added existingCves and existingCvesUrl fields to represent CVEs currently present on operating systems, along with their source URLs. These fields include validation rules for patterns and maximum lengths.
  • Introduced fixedCves and fixedCvesUrl fields to describe CVEs fixed by a specific OS version, with similar validation rules.
  • Added existingCves field to represent the CVEs currently present in the Instance

ITEP-67266

Any Newly Introduced Dependencies

Please describe any newly introduced 3rd party dependencies in this change. List their name, license information and how they are used in the project.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes
  • I have not introduced any 3rd party dependency changes
  • I have performed a self-review of my code

Copy link
Contributor

@SushilLakra SushilLakra left a comment

Choose a reason for hiding this comment

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

LGTM

@daniele-moro daniele-moro enabled auto-merge (squash) May 28, 2025 16:54
@daniele-moro daniele-moro merged commit 9ac302e into main May 28, 2025
20 checks passed
@daniele-moro daniele-moro deleted the cve_api_changes branch May 28, 2025 17:08
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.

6 participants