Skip to content

[Cycode] Fix for vulnerable manifest file dependency - golang.org/x/crypto updated to version 0.52.0 - #107

Open
cycode-security[bot] wants to merge 1 commit into
mainfrom
cycode-fix-suggestion-manifest-dependency-update-fbb3a9e7-85e9-4bb2-9fe6-3a4fe6797b26
Open

[Cycode] Fix for vulnerable manifest file dependency - golang.org/x/crypto updated to version 0.52.0#107
cycode-security[bot] wants to merge 1 commit into
mainfrom
cycode-fix-suggestion-manifest-dependency-update-fbb3a9e7-85e9-4bb2-9fe6-3a4fe6797b26

Conversation

@cycode-security

@cycode-security cycode-security Bot commented Jul 7, 2026

Copy link
Copy Markdown

Cycode Vulnerable Dependencies Update

This pull request updates the following manifest file:

File Path Number of packages to update
go.mod 1

📂 go.mod

1 package will be updated to resolve vulnerabilities:

Package Name Current Version Updated Version
golang.org/x/crypto 0.41.0 0.52.0

Note

Medium Risk
Touches a crypto library used for password hashing; risk is moderate due to dependency churn, but scope is limited to a version bump with no logic changes.

Overview
Bumps the direct golang.org/x/crypto dependency in go.mod from 0.41.0 to 0.52.0 to address reported vulnerabilities (Cycode manifest scan). There are no application code changes in this PR.

That module is used in-repo for bcrypt password hashing and comparison (internal/api/crypto/hash.go), so the upgrade is security-relevant even though behavior should stay the same aside from patched crypto implementations.

Reviewed by Cursor Bugbot for commit e05173f. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit e05173f. Configure here.

Comment thread go.mod
github.com/tidwall/gjson v1.18.0
github.com/zitadel/oidc/v3 v3.44.0
golang.org/x/crypto v0.41.0
golang.org/x/crypto v0.52.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Missing go.sum crypto entries

High Severity

The direct require for golang.org/x/crypto is raised to v0.52.0, but root go.sum still only records checksums for v0.41.0. That leaves the module graph out of sync, so go mod verify and builds that enforce recorded sums can fail until go.sum is updated.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e05173f. Configure here.

Comment thread go.mod
github.com/tidwall/gjson v1.18.0
github.com/zitadel/oidc/v3 v3.44.0
golang.org/x/crypto v0.41.0
golang.org/x/crypto v0.52.0

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Crypto bump needs Go 1.25

Medium Severity

golang.org/x/crypto v0.52.0 declares go 1.25.0, while this module still specifies go 1.24.0 and toolchain go1.24.2. On toolchains with GOTOOLCHAIN=local, go mod tidy and builds can fail until the Go/toolchain version is raised or auto toolchain is allowed.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit e05173f. Configure here.

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.

0 participants