Skip to content

Conversation

@AkihiroSuda
Copy link
Member

Silence the following govulncheck reports
("you import and 3 vulnerabilities in modules you require, but your code doesn't appear to call these vulnerabilities"):

Vulnerability #1: GO-2025-4135
    Malformed constraint may cause denial of service in
    golang.org/x/crypto/ssh/agent
  More info: https://pkg.go.dev/vuln/GO-2025-4135
  Module: golang.org/x/crypto
    Found in: golang.org/x/crypto@v0.41.0
    Fixed in: golang.org/x/crypto@v0.45.0

Vulnerability #2: GO-2025-4134
    Unbounded memory consumption in golang.org/x/crypto/ssh
  More info: https://pkg.go.dev/vuln/GO-2025-4134
  Module: golang.org/x/crypto
    Found in: golang.org/x/crypto@v0.41.0
    Fixed in: golang.org/x/crypto@v0.45.0

Vulnerability #3: GO-2025-4116
    Potential denial of service in golang.org/x/crypto/ssh/agent
  More info: https://pkg.go.dev/vuln/GO-2025-4116
  Module: golang.org/x/crypto
    Found in: golang.org/x/crypto@v0.41.0
    Fixed in: golang.org/x/crypto@v0.43.0

This commit also drops the support for Go 1.23, and cherry-picks:

@github-project-automation github-project-automation bot moved this to Needs Triage in Pull Request Review Dec 8, 2025
@dosubot dosubot bot added the dependencies Pull requests that update a dependency file label Dec 8, 2025
Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@estesp
Copy link
Member

estesp commented Dec 8, 2025

Sorry, can you rebase on main after the Go updates backported from the main/2.x branches?

@github-project-automation github-project-automation bot moved this from Needs Triage to Review In Progress in Pull Request Review Dec 8, 2025
module github.com/containerd/containerd

go 1.23.0
go 1.24.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There was an earlier discussion on changing the minimum go version on an existing release branch and we didnt want to do this bump cc: @thaJeztah

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is fine since 1.23 is not in support and this is a transitive requirement. So if we accept the v0.45.0 version, we are also accepting the 1.24 minimum.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed with Mike on this, updating the minimum version will be the best way forward.

mxpv and others added 3 commits December 9, 2025 17:02
Updates BUILDING.md to specify that the project supports the two most recent major Go versions,
rather than hardcoding "1.22.x or above" (as we don't support 1.22).

This ensures the documentation doesn't become outdated when new Go versions are released.

Signed-off-by: Maksym Pavlenko <pavlenko.maksym@gmail.com>
(cherry picked from commit fb28794)
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
@dmcgowan dmcgowan merged commit 3b655c2 into containerd:release/1.7 Dec 9, 2025
49 checks passed
@github-project-automation github-project-automation bot moved this from Review In Progress to Done in Pull Request Review Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file size/XXL

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

8 participants