Skip to content

Use go.uber.org/mock for tests - #401

Closed
atanzu wants to merge 1 commit into
masterfrom
mark.kirichenko/add-uber-mock
Closed

atanzu wants to merge 1 commit into
masterfrom
mark.kirichenko/add-uber-mock

Conversation

@atanzu

@atanzu atanzu commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Replace archived github.com/golang/mock with go.uber.org/mock. longer maintained.

Transitive dependencies of golang/mock pull in old, vulnerable versions of golang.org/x/crypto and golang.org/x/net that trigger security scanners.

Resolves: #394

Replace archived `github.com/golang/mock` with `go.uber.org/mock`.
longer maintained.

Transitive dependencies of `golang/mock` pull in old, vulnerable
versions of `golang.org/x/crypto` and `golang.org/x/net` that trigger
security scanners.

Signed-off-by: Mark Kirichenko <mark.kirichenko@datadoghq.com>
@datadog-prod-us1-4

Copy link
Copy Markdown

Code Coverage

🛑 Gate Violations

🎯 1 Code Coverage issue detected

A Patch coverage percentage gate may be blocking this PR.

• Patch coverage: no data available (threshold: 90.00%)

ℹ️ Info

🎯 Code Coverage (details)
• Patch Coverage: No data available
• Overall Coverage: 84.70% (-0.35%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 0745458 | Docs | View more details | Give us feedback!

@atanzu atanzu closed this Aug 25, 2026
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.

github.com/golang/mock contain vulnerable crypto package

1 participant