-
Notifications
You must be signed in to change notification settings - Fork 573
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
Replace grype db diff
with grype db search --since DATE
#2129
Comments
hi! @wagoodman diff is important since we want to track changes of the DB. |
As eluded to in #2128 (comment) there is a way to compute the records that differ. Is this an important use case to you? If so, can you tell us more about why so? We dropped figuring this out in v6 to cut scope, but if we have enough users that really depend on this then that would be good reason to keep it in scope (in which case I'd close this issue and make a new one on how to implement the diff command). |
I use this command to handle states of last-modified CVEs. |
@TimBrown1611 when you say some sources don't provide modified or created date, do you mean in the upstream data? Like, are there specific data sources that grype db consumes that are missing this data? If so, which ones? If grype had a date added and date updated date for most issues in each database, would you still need |
From discussion on our livestream on 10 October 2024, we had the idea: Can we add Things to consider:
|
diff
commandgrype db diff
with grype db search --since DATE
Hi,
|
As currently written the DB diff command will be cumbersome to keep as a feature while working on v6. We could leave DB diff in tact for v1-5 schemas, but this would be confusing for users attempting to use this diff feature for the latest (v6) DBs.
V6 currently is going to have a blobs table where digests for content could be recomputed and used as a foundation for a diff, however that is out of scope for v6 unless there is a convincing argument / need to keep this feature and add to the v6 scope further.
The text was updated successfully, but these errors were encountered: