Fix: make failed CPE validation correctly return error #2762
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Previously, this method incorrectly return an empty Attributes object and a nil error, leading to callers attempting to use the empty attributes object. Signed-off-by: Will Murphy <[email protected]>
* main: (343 commits) feat: update haproxy classifier (#3277) chore(deps): update tools to latest versions (#3291) fix: don't use builtin scanner in licensecheck (#3290) chore(deps): update CPE dictionary index (#3288) chore(deps): bump github/codeql-action from 3.26.9 to 3.26.10 (#3289) update redis classifier (#3281) fix: improve node classifier version matching (#3284) fix: update ruby classifier for -rc, -dev, etc. versions (#3285) chore(deps): update CPE dictionary index (#3262) chore(deps): bump github.com/docker/docker (#3264) chore(deps): bump github/codeql-action from 3.26.8 to 3.26.9 (#3275) chore(deps): update stereoscope to dc10ea61fd18efa45b516eda4de8bc19d8322429 (#3280) chore(deps): bump actions/checkout from 4.1.7 to 4.2.0 (#3283) add awaiting response management (#3272) fix: correct excluded mount point comparison to file paths (#3269) Add JVM cataloger (#3217) feat: classifier for Dart lang binaries (#3265) Add compliance policy for empty name and version (#3257) chore(deps): bump github.com/github/go-spdx/v2 from 2.3.1 to 2.3.2 (#3254) chore(deps): bump peter-evans/create-pull-request from 7.0.3 to 7.0.5 (#3255) ...
Signed-off-by: Christopher Phillips <[email protected]>
Signed-off-by: Christopher Phillips <[email protected]>
|
@willmurphyscode - I merged this with upstream and found the NewAttributes case you mentioned in TODO and added a test. I think this is ready for review. If you 🟢 my follow up work I 🟢 the work that come before and we can merge as |
spiffcs
approved these changes
Oct 2, 2024
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously, this method incorrectly return an empty Attributes object and a nil error, leading to callers attempting to use the empty attributes object.
We need to be careful with this, because turning on validation that previously had no effect is always a little scary. Creating for discussion for now.
This is one aspect of the cause of anchore/grype#1792
TODO:
NewAttributesand make sure they have the right behavior when it returns an error; someone might be unintentionally counting on the error being nil.