We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The current models is that a Package is related to Vulnerability through a generic PackageRelatedVulnerability relationship with a fix attribute
Package
Vulnerability
PackageRelatedVulnerability
fix
https://github.com/nexB/vulnerablecode/blob/40a39743f385bd5b6dfa3424bc72231fe1ae7456/vulnerabilities/models.py#L491
https://github.com/nexB/vulnerablecode/blob/40a39743f385bd5b6dfa3424bc72231fe1ae7456/vulnerabilities/models.py#L584
https://github.com/nexB/vulnerablecode/blob/40a39743f385bd5b6dfa3424bc72231fe1ae7456/vulnerabilities/models.py#L614
This approach is problematic and not obvious. It makes queries more complex and slower.
We should instead evolve the models towards separate AffectedPackages and FixingPackage or something along these lines to be designed.
See these related issues:
AffectedPackage
The text was updated successfully, but these errors were encountered:
@pombredanne
It makes queries more complex
We could possibly avoid that by some abstraction as proposed in #595
and slower
Slower than ... ?
Sorry, something went wrong.
As per discussion during call:
A fix is more complex than just a package. It can be much more than that. Fixes can come in terms of fix commits, individual patches etc @pombredanne
TG1999
No branches or pull requests
The current models is that a
Package
is related toVulnerability
through a genericPackageRelatedVulnerability
relationship with afix
attributehttps://github.com/nexB/vulnerablecode/blob/40a39743f385bd5b6dfa3424bc72231fe1ae7456/vulnerabilities/models.py#L491
https://github.com/nexB/vulnerablecode/blob/40a39743f385bd5b6dfa3424bc72231fe1ae7456/vulnerabilities/models.py#L584
https://github.com/nexB/vulnerablecode/blob/40a39743f385bd5b6dfa3424bc72231fe1ae7456/vulnerabilities/models.py#L614
This approach is problematic and not obvious. It makes queries more complex and slower.
We should instead evolve the models towards separate AffectedPackages and FixingPackage or something along these lines to be designed.
See these related issues:
AffectedPackage
as model #727The text was updated successfully, but these errors were encountered: