Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: improve go binary semver extraction for traefik #3325

Merged
merged 1 commit into from
Oct 14, 2024

Conversation

westonsteimel
Copy link
Contributor

Enhances the go cataloger semver extraction logic to include getting the release version of traefik. This is based off of the regex pattern that already existed in the traefik binary classifier.

The advantage to raising up the release version in the go cataloger rather than just in the binary classifier is that it means the vulnerability matching can take advantage of the GitHub security advisory data for traefik rather than just the NVD CVE entries. I think this would have prevented anchore/grype#2178 from occurring

Improves the go cataloger semver extraction logic to include getting the
release version of traefik.  This is based off of the regex pattern that
already existed in the traefik binary classifier.

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
@westonsteimel westonsteimel force-pushed the improve-go-binary-semver-file-string-extraction branch from 7f31887 to 14c0fe7 Compare October 14, 2024 11:13
@westonsteimel westonsteimel requested a review from a team October 14, 2024 11:37
@westonsteimel westonsteimel enabled auto-merge (squash) October 14, 2024 11:37
@westonsteimel westonsteimel merged commit e962c10 into main Oct 14, 2024
12 checks passed
@westonsteimel westonsteimel deleted the improve-go-binary-semver-file-string-extraction branch October 14, 2024 13:41
@wagoodman wagoodman added the bug Something isn't working label Oct 14, 2024
HeyeOpenSource pushed a commit to HeyeOpenSource/syft that referenced this pull request Oct 14, 2024
Improves the go cataloger semver extraction logic to include getting the
release version of traefik.  This is based off of the regex pattern that
already existed in the traefik binary classifier.

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
Signed-off-by: HeyeOpenSource <opensource@heye-international.com>
HeyeOpenSource pushed a commit to HeyeOpenSource/syft that referenced this pull request Oct 14, 2024
Improves the go cataloger semver extraction logic to include getting the
release version of traefik.  This is based off of the regex pattern that
already existed in the traefik binary classifier.

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
Signed-off-by: HeyeOpenSource <opensource@heye-international.com>
HeyeOpenSource pushed a commit to HeyeOpenSource/syft that referenced this pull request Oct 15, 2024
Improves the go cataloger semver extraction logic to include getting the
release version of traefik.  This is based off of the regex pattern that
already existed in the traefik binary classifier.

Signed-off-by: Weston Steimel <commits@weston.slmail.me>
Signed-off-by: HeyeOpenSource <opensource@heye-international.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants