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: improve determinism in java archive identification #3085

Conversation

kzantow
Copy link
Contributor

@kzantow kzantow commented Jul 30, 2024

This PR fixes a case where Syft is generating Java packages in a nondeterministic manner when there are multiple pom.properties that it considers to be "matching".

This PR is only to make a deterministic SBOM, and we can iterate on this later to further improve identification, such as: #3054

Partially fixes: #2967

Signed-off-by: Keith Zantow <kzantow@gmail.com>
Signed-off-by: Keith Zantow <kzantow@gmail.com>
@kzantow kzantow merged commit a4b5dcd into anchore:main Jul 30, 2024
11 checks passed
@kzantow kzantow deleted the fix/deterministic-java-pom-properties-order branch July 30, 2024 16:02
westonsteimel added a commit that referenced this pull request Aug 9, 2024
Updates the purl generation tests now that syft has better deterministic
results per #3085

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
westonsteimel added a commit that referenced this pull request Aug 9, 2024
Updates the purl generation tests now that syft has more deterministic
results per #3085

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
westonsteimel added a commit that referenced this pull request Aug 9, 2024
Updates the purl generation tests now that syft has more deterministic
results per #3085

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
westonsteimel added a commit that referenced this pull request Aug 9, 2024
Updates the purl generation tests now that syft has more deterministic
results per #3085

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
westonsteimel added a commit that referenced this pull request Aug 9, 2024
Updates the purl generation tests now that syft has more deterministic
results per #3085

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
westonsteimel added a commit that referenced this pull request Aug 9, 2024
ensures correct package url generation for more java packages now that
syft has more deterministic results per #3085

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
westonsteimel added a commit that referenced this pull request Aug 9, 2024
ensures correct package url generation for more java packages now that
syft has more deterministic results per #3085

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
westonsteimel added a commit that referenced this pull request Aug 9, 2024
ensures correct package url generation for more java packages now that
syft has more deterministic results per #3085

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
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.

Order CPEs deterministically for SBOM reproducibility
2 participants