Skip to content

[K9VULN-5229] Upgrade packageurl-go to v0.1.3#97

Merged
rjcoulter22 merged 2 commits intomainfrom
rjcoulter/k9vuln-5229
Jan 9, 2026
Merged

[K9VULN-5229] Upgrade packageurl-go to v0.1.3#97
rjcoulter22 merged 2 commits intomainfrom
rjcoulter/k9vuln-5229

Conversation

@rjcoulter22
Copy link
Copy Markdown
Contributor

@rjcoulter22 rjcoulter22 commented Jan 9, 2026

🚀 Motivation

This package we downgraded hereThe for the intents of better comparing the sbom-generator results to the deprecated osv-scanner. Specifically the downgraded version gets rid of some character escaping that was being done for PURLs which was contributing to discrepancies in the results of the two scanners.

📚 Documentation

Document Link or Detail
RFC N/A
Incident N/A
Jira Ticket K9VULN-5229 - PURL encoding issue

📝 Summary

Updated packageurl-go from v0.1.1 to v0.1.3 to ensure proper URL encoding of special characters in Package URLs (PURLs). The newer version correctly encodes characters like + (%2B), : (%3A), @ (%40), and $ (%24) according to the PURL specification. This ensures consistency with third-party tools like Snyk when processing SBOMs.

You can see the releases of this library and the associated changes here https://github.com/package-url/packageurl-go/releases

🧪 Testing

  • [] New tests were added for new logic.
  • Existing tests were updated for new logic, and not only so that they pass!
  • Benchmark results prove that performance is the same or better.

🚧 Staging validation

  • Deployed and monitored using Datadog dashboards.
  • Proof that it works as expected, including profiling or UX screenshots.

🆘 Recovery

Notes for on-call - select only one:

  • The change can be rolled back.
  • Do not roll back. Why?:

@rjcoulter22 rjcoulter22 changed the title Rjcoulter/k9vuln 5229 [K9VULN-5229] Upgrade packageurl-go to v0.1.3 Jan 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 9, 2026

Go test coverage report

Total test coverage: 90.2% (4271/4733)

No coverage changes in any files compared to the base.

@rjcoulter22 rjcoulter22 marked this pull request as ready for review January 9, 2026 16:39
@rjcoulter22 rjcoulter22 requested a review from a team as a code owner January 9, 2026 16:39
@rjcoulter22 rjcoulter22 merged commit 52249c6 into main Jan 9, 2026
10 checks passed
@rjcoulter22 rjcoulter22 deleted the rjcoulter/k9vuln-5229 branch January 9, 2026 17:24
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.

2 participants