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

Keep content in rejected CVE Record #6

Open
zmanion opened this issue Dec 7, 2023 · 2 comments
Open

Keep content in rejected CVE Record #6

zmanion opened this issue Dec 7, 2023 · 2 comments

Comments

@zmanion
Copy link
Collaborator

zmanion commented Dec 7, 2023

Currently when a record is rejected, the description is replaced with template text and sometimes a brief explanation. A rejected record uses different JSON (cnaRejectedContainer) and any content in a published record (cnaPublishedContainer) is lost. adpContainer would be lost too. History might be retained in the cvelist GitHub respository.

Consider in the future keeping any existing content when rejecting a record. This requires both non trivial JSON schema changes and services/backend changes.

@zmanion
Copy link
Collaborator Author

zmanion commented Apr 24, 2024

@zmanion
Copy link
Collaborator Author

zmanion commented Aug 27, 2024

This remains an issue, I (and I expect others) could have used the original information in CVE-2024-42992. Changing the schema for rejected IDs removes useful information. This becomes more of a concern as ADP containers will be removed also.

Proposal for rejecting a record

In the schema/services:

  1. Set status to rejected
  2. Require an explanation field
  3. Store the rejected datetime
  4. Do not modify any other content

In the cve.org rendering:

  1. Prominently convey rejected status and the explanation, perhaps even hide the original content/containers until the user twists them open

In policy:

  1. Require an explanation
  2. Require the Secretariat and CNA-LR (and maybe others?) to prominently convey rejected status when rendering CVE Record content, similar to

4.5.3.11 CVE Program members MUST indicate “disputed” status and present the explanation when rendering and displaying CVE Record data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant