-
Notifications
You must be signed in to change notification settings - Fork 416
skaffold/2.17.0-r2: cve remediation #75230
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
skaffold/2.17.0-r2: cve remediation #75230
Conversation
<!--ci-cve-scan:must-fix: GHSA-4qg8-fj49-pxjh-->
🔢 Build Failed: Dependency Version Mismatch
Build Details
Root Cause Analysis 🔍The go.mod file for github.com/sigstore/timestamp-authority at version v2.0.3 declares itself as module path 'github.com/sigstore/timestamp-authority/v2', but the dependency is being referenced as 'github.com/sigstore/[email protected]' without the '/v2' suffix. This violates Go module versioning rules where major version 2+ must include the version in the module path. 🔍 Build failure fix suggestionsFound similar build failures that have been fixed in the past and analyzed them to suggest a fix: Similar PRs with fixesSuggested ChangesFile: skaffold.yaml
Replacement: Content: Click to expand fix analysisAnalysisThe similar fixes show a clear pattern: when Go modules use v2+ versioning, they require the version suffix in the module path. Both containerd fixes demonstrate the same approach - adding dual dependency entries: one for the v1.x version using the original module path (github.com/containerd/[email protected]) and another for the v2.x version using the versioned module path (github.com/containerd/containerd/[email protected]). This allows the build system to properly resolve dependencies that may require either version format. Click to expand fix explanationExplanationThe fix works because Go's module system requires that for major versions 2 and higher, the module path must include the version suffix (e.g., /v2). The timestamp-authority project at v2.0.3 has declared its module path as 'github.com/sigstore/timestamp-authority/v2' in its go.mod file, but the dependency is being referenced as 'github.com/sigstore/[email protected]' without the '/v2' suffix. By changing the dependency reference to include the correct module path 'github.com/sigstore/timestamp-authority/[email protected]', the Go module system will be able to properly resolve and fetch the dependency. This aligns with Go's semantic import versioning rules and matches the pattern seen in the containerd fixes where the v2+ versions required the versioned module path. Click to expand alternative approachesAlternative Approaches
Was this comment helpful? Please use 👍 or 👎 reactions on this comment. |
|
Advisory PR: wolfi-dev/advisories#27919 |
|
This vulnerability remediation is stale and no longer needed. 👋 Advisory CGA-44vc-cjxq-2296 has the latest event type of "pending-upstream-fix": https://github.com/wolfi-dev/advisories/blob/main/skaffold.advisories.yaml |
skaffold/2.17.0-r2: fix GHSA-4qg8-fj49-pxjh
Advisory data: https://github.com/wolfi-dev/advisories/blob/main/skaffold.advisories.yaml
"Breadcrumbs" for this automated service