Skip to content

Debian import considers status - #60

Merged
pombredanne merged 5 commits into
aboutcode-org:developfrom
haikoschol:import-considers-status
Sep 26, 2019
Merged

Debian import considers status#60
pombredanne merged 5 commits into
aboutcode-org:developfrom
haikoschol:import-considers-status

Conversation

@haikoschol

Copy link
Copy Markdown
Collaborator

This fixes #38 for Debian. The current data source for Ubuntu does not contain package versions and is therefore not very useful. I created #59 to address this.

The previous fixture had some strange values like "not-resolved"
for "status" and "un-remote" for "scope". It doesn't look like
those values are used by the Debian security tracker.
To address issue aboutcode-org#38 we need records with "status" values
"resolved" and "open".

Signed-off-by: Haiko Schol <hs@haikoschol.com>
Before this change, only "fixed_version" was retained. This is
the lowest version of the package that includes a fix for the
given vulnerability. However in the meantime, newer versions
of the package might have become available in the Debian release
passed in as "base_release".

Signed-off-by: Haiko Schol <hs@haikoschol.com>
Signed-off-by: Haiko Schol <hs@haikoschol.com>
Signed-off-by: Haiko Schol <hs@haikoschol.com>
Currently only two status values are recognized:
"open" maps to ImpactedPackage
"resolved" maps to ResolvedPackage

Signed-off-by: Haiko Schol <hs@haikoschol.com>

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. 👍

@pombredanne
pombredanne merged commit af561ae into aboutcode-org:develop Sep 26, 2019
@haikoschol
haikoschol deleted the import-considers-status branch April 10, 2023 00:44
pombredanne added a commit that referenced this pull request Apr 2, 2025
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

Successfully merging this pull request may close these issues.

Import of Debian and Ubuntu packages ignores status field

2 participants