-
Notifications
You must be signed in to change notification settings - Fork 201
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
Match affected and fixed-by Packages #1528
Conversation
Reference: #1287 Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Reference: #1287 Signed-off-by: John M. Horan johnmhoran@gmail.com
@johnmhoran thanks! please provide screenshots and let us know when it's good to be merged |
Reference: #1287 - Matched affected/fixed-by Package data added to Vulnerability details. - version_class-based sorting added to that data as well. - Note that version_class-based Packages sorting still needs to be added to Package search results. - Filter added to encode Package URLs. - Next: will add tests. Signed-off-by: John M. Horan <johnmhoran@gmail.com>
…m:nexB/vulnerablecode into 1287-resume-combining-affected-fixed-tabs
@TG1999 I just committed and tried to push my latest changes -- no tests yet, and I'm not yet able to properly sort the Package search results (but the immediate task, the new matched affected/fixed by records, look great and are properly sorted ;-) -- but immediately ran into a problem evidently because you had committed and pushed numerous changes of your own to this branch that I did not have locally. I followed the git instruction to run 'git pull' on this branch, it seems to have been merged automatically and without reported conflicts. I then ran |
Reference: #1287 - Fixed merge conflicts. Signed-off-by: John M. Horan <johnmhoran@gmail.com>
@TG1999 I just updated The primary goal of this issue and PR was matching the affected and fixed-by Packages, and that's now in place on the Vulnerability details page. (I'll add a screen shot or two in another comment below.) My next task will be to add some tests. This PR also includes a few other items I encountered along the way:
Note: I also tried to update the PackageSearch class in views.py to incorporate my version_class-based sorting into the |
@TG1999 A few screen shots of the new matched affect/fixed-by Package records on the Vulnerabilities details page. The landing page showing the Scrolling down the And this is the new |
@TG1999 @pombredanne While exploring the sorting of a variety of PURL types using Although Not sure if this is still needed (or relevant to my question), but we import
|
Note that alpine_test.txt.ABOUT contains a reference to an
but |
I'm seeing similar key errors for some other PURL types like |
@TG1999 @pombredanne This PR is ready for review. I've refactored the sort and version-class functions and added tests for these functions and for the custom quote filter. Note: After learning that not all "known PURL types" -- https://github.com/package-url/purl-spec/blob/master/PURL-TYPES.rst#known-purl-types -- have a univers version_class, I revised the sort and version-class tests to test only PURL types currently supported by univers. I see that once again the only failing check here is |
Reference: #1287 Signed-off-by: John M. Horan <johnmhoran@gmail.com>
Thanks @johnmhoran ! LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
The initial commit in this PR adds CSS for the collapsed (< 1024px) responsive Bulma navbar and adjusts some template nesting. Substantive work on matching affected and fixed-by Packages to follow.
Reference: #1287