Skip to content

Conversation

@octo-sts
Copy link
Contributor

@octo-sts octo-sts bot commented Mar 12, 2025

datadog-agent/7.63.3-r1: fix CVE-2025-22868

Advisory data: https://github.com/wolfi-dev/advisories/blob/main/datadog-agent.advisories.yaml

Source code for this service: https://go/cve-remedy-automation-source

Logs for this execution: https://go/cve-remedy-automation-logs

Docs for this service: (not provided yet)

@octo-sts octo-sts bot added P1 This label indicates our scanning found High, Medium or Low CVEs for these packages. automated pr CVE-2025-22868 go/bump request-cve-remediation labels Mar 12, 2025
@octo-sts
Copy link
Contributor Author

octo-sts bot commented Mar 12, 2025

⚠️ EXPERIMENTAL

Please use 👍 or 👎 on this comment to indicate if you agree or disagree with the recommendation.

To provide more detailed feedback please comment on the recommendation prefixed with /ai-verify:

e.g. /ai-verify partially helpful but I also added bash to the build environment

Gen AI suggestions to solve the build error:

• Detected Error: "Error: failed to run update. Error: package golang.org/x/oauth2 was not found on the go.mod file. Please remove the package or add it to the list of 'replaces'"

• Error Category: Build/Dependency

• Failure Point: go/bump step trying to update golang.org/x/oauth2 in pkg/trace/go.mod

• Root Cause Analysis: The go/bump step is trying to update golang.org/x/oauth2 but this package isn't directly declared in the pkg/trace/go.mod file, it's likely a transitive dependency.

• Suggested Fix: Modify the go/bump step in the melange.yaml to only update the packages that are directly declared:

- uses: go/bump
  with:
    modroot: pkg/trace
    deps: |-
      github.com/docker/[email protected]+incompatible
      golang.org/x/[email protected]
    show-diff: true

• Explanation: By removing golang.org/x/oauth2 from the deps list, we only update packages that are directly declared in the go.mod file. The oauth2 package will still be updated through the normal module dependency resolution process.

• Additional Notes:

  • The error is specific to the pkg/trace/go.mod file
  • Transitive dependencies should be handled by Go's module system automatically
  • The +incompatible suffix for docker/distribution is correctly maintained

• References:

@octo-sts octo-sts bot added the ai/skip-comment Stop AI from commenting on PR label Mar 12, 2025
@octo-sts octo-sts bot added the bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. label Mar 13, 2025
@AmberArcadia AmberArcadia requested a review from a team March 13, 2025 20:55
@powersj powersj merged commit c91a297 into main Mar 13, 2025
20 of 21 checks passed
@powersj powersj deleted the cve-datadog-agent-ecccdf81abf0059ddc029980060280cc branch March 13, 2025 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai/skip-comment Stop AI from commenting on PR automated pr bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. CVE-2025-22868 go/bump manual/review-needed P1 This label indicates our scanning found High, Medium or Low CVEs for these packages. request-cve-remediation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants