-
Notifications
You must be signed in to change notification settings - Fork 416
falcoctl/0.11.4-r3: cve remediation #74957
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
falcoctl/0.11.4-r3: cve remediation #74957
Conversation
<!--ci-cve-scan:must-fix: GHSA-4qg8-fj49-pxjh-->
🔢 Build Failed: Dependency Version Mismatch
Build Details
Root Cause Analysis 🔍The gobump tool attempted to update the github.com/sigstore/timestamp-authority dependency to v2.0.3, but this version has an incompatible module path. The go.mod file at that revision expects the module path to be 'github.com/sigstore/timestamp-authority/v2' (with /v2 suffix) for v2+ versions according to Go module versioning conventions, but the dependency is being referenced without the /v2 suffix. 🔍 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: falcoctl.yaml
Replacement: Content: Click to expand fix analysisAnalysisBoth similar fixes followed the same pattern: when Go modules have post-v2 module paths (with /v2 suffix), the dependency specification needs to include both the original module path for v1.x versions and the versioned module path for v2+ versions. In both cases, the fix involved adding two dependency entries in the go/bump step - one for the v1.x branch (e.g., github.com/containerd/[email protected]) and another for the v2+ branch with the /v2 suffix (e.g., github.com/containerd/containerd/[email protected]). This addresses Go's module versioning convention where major version 2+ requires the version suffix in the module path. Click to expand fix explanationExplanationThe current build failure occurs because github.com/sigstore/[email protected] has a go.mod file that specifies the module path as 'github.com/sigstore/timestamp-authority/v2' (with /v2 suffix) to comply with Go's semantic import versioning. When referencing v2+ versions of Go modules, the import path must include the major version suffix. By changing the dependency specification from 'github.com/sigstore/[email protected]' to 'github.com/sigstore/timestamp-authority/[email protected]', we align with the module's actual path declaration in its go.mod file, which resolves the version validation error. Click to expand alternative approachesAlternative Approaches
Was this comment helpful? Please use 👍 or 👎 reactions on this comment. |
github.com/sigstore/timestamp-authority has modified their module naming to include `v2`. Adapt the bump to use the new v2 version. Signed-off-by: David Negreira <[email protected]>
Signed-off-by: David Negreira <[email protected]>
b39eccf to
a94bd70
Compare
|
needs advisory wolfi-dev/advisories#27905 |
|
This vulnerability remediation is stale and no longer needed. 👋 Advisory CGA-85q9-vm8j-j4g5 has the latest event type of "pending-upstream-fix": https://github.com/wolfi-dev/advisories/blob/main/falcoctl.advisories.yaml |
falcoctl/0.11.4-r3: fix GHSA-4qg8-fj49-pxjh
Advisory data: https://github.com/wolfi-dev/advisories/blob/main/falcoctl.advisories.yaml
The following vulnerabilities are being deferred to future PRs (to avoid merge conflicts):
github.com/sigstore/[email protected]/sigstore/[email protected]"Breadcrumbs" for this automated service