Skip to content
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

Fix file metadata cataloger when passed explicit coordinates #2370

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

wagoodman
Copy link
Contributor

There are two reasons why the file metadata cataloger is not working today when provided explicit file coordinates:

  • the channel was immediately closed, so no results could have ever been provided
  • we trust the user-provided coordinates exist in the resolver. This can cause a few issues relating to the stereoscope file reference object not being available (or the coordinates are not correct). Either way, the coordinates need to be resolved to a full location object.

Note: this execution path is not used since it is only possible to catalog all locations today (instead of a certain subset, which when done these issues start to be noticed).

(This was broken off from #1383 )

Signed-off-by: Alex Goodman <wagoodman@users.noreply.github.com>
@wagoodman wagoodman added the bug Something isn't working label Nov 28, 2023
@wagoodman wagoodman marked this pull request as ready for review November 28, 2023 21:57
@wagoodman wagoodman requested a review from a team November 28, 2023 21:58
@wagoodman wagoodman enabled auto-merge (squash) November 28, 2023 22:46
@wagoodman wagoodman merged commit 5c8dd4c into main Nov 29, 2023
10 checks passed
@wagoodman wagoodman deleted the use-resolver-locations branch November 29, 2023 14:42
spiffcs added a commit to coheigea/syft that referenced this pull request Dec 5, 2023
* main: (40 commits)
  chore(deps): bump anchore/sbom-action from 0.15.0 to 0.15.1 (anchore#2392)
  Retrieve remote licenses using pom.properties when there is no pom.xml (anchore#2315)
  fix(java): improve identification for org.apache.tapestry artifacts (anchore#2384)
  fix(java): improve identification for io.ratpack artifacts (anchore#2379)
  fix(java): improve identification for org.apache.cassandra artifacts (anchore#2386)
  fix(java): improve identification for org.neo4j.procedure artifacts (anchore#2388)
  fix: bump fangs for ptr summarize fix (anchore#2387)
  fix(java): improve identification for org.elasticsearch artifacts (anchore#2383)
  fix(java): improve identification for org.apache.geode artifacts (anchore#2382)
  fix(java): improve identification for org.apache.tomcat.embed artifacts (anchore#2381)
  fix(java): improve identification for io.projectreactor.netty artifacts (anchore#2378)
  fix(java): improve identification for org.eclipse.platform artifacts (anchore#2349)
  Generalize UI events for cataloging tasks (anchore#2369)
  chore(deps): update tools to latest versions (anchore#2376)
  chore(deps): bump github.com/google/go-containerregistry (anchore#2377)
  chore: fix tests failing due to Mac Rosetta cache (anchore#2374)
  fix: improve dotnet portable executable identification (anchore#2133)
  fix file metadata cataloger to use resolved locations (anchore#2370)
  fix: logging level for parsing potential PE files (anchore#2367)
  only remove breaking-change label when there are schema changes (anchore#2371)
  ...

Signed-off-by: Christopher Phillips <christopher.phillips@anchore.com>
GijsCalis pushed a commit to GijsCalis/syft that referenced this pull request Feb 19, 2024
Signed-off-by: Alex Goodman <wagoodman@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants