Skip to content
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

CVE-2025-22869 golang.org/x/crypto bump version #2270

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

irlndts
Copy link

@irlndts irlndts commented Mar 5, 2025

Hi, there is a vulnerability CVE-2025-22869

I know pgx actually doesn't use the vulnerable functionality in golang.org/x/crypto but scanner alert still works.

I've done followings:

  • Update go.mod to bump crypto version to v0.35.0
  • Update go version to 1.24 since crypto v0.35.0 requires it
  • Then go mod tidy
  • Then go test ./... (everything is fine)

@jackc
Copy link
Owner

jackc commented Mar 15, 2025

Are you sure about the Go 1.24 requirement? It looks like 1.23 is sufficient to me. https://cs.opensource.google/go/x/crypto/+/master:go.mod

I'd like to avoid that change as that would break pgx's Go compatible versions policy.

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