Skip to content

fix(deps): BDSA-2026-1849 cosign transient dependency pinning#1858

Closed
matthiasbruns wants to merge 2 commits intoopen-component-model:mainfrom
matthiasbruns:deps/BDSA-2026-1849
Closed

fix(deps): BDSA-2026-1849 cosign transient dependency pinning#1858
matthiasbruns wants to merge 2 commits intoopen-component-model:mainfrom
matthiasbruns:deps/BDSA-2026-1849

Conversation

@matthiasbruns
Copy link
Copy Markdown
Contributor

@matthiasbruns matthiasbruns commented Mar 10, 2026

What this PR does / why we need it

BlackDuck found a CVE in mongodb/mongo-go-driver@v1.17.6 which is used in cosign@v3.0.5

I already created a PR in upstream sigstore/cosign#4764

Changelog

https://github.com/mongodb/mongo-go-driver/releases/tag/v1.17.9
🐛 Fixed
GODRIVER-3793 Fix variable shadowing in rtt monitor in mongodb/mongo-go-driver#2317

https://github.com/mongodb/mongo-go-driver/releases/tag/v1.17.8
✨ New Features
GODRIVER-3773 Deprecate v1 in mongodb/mongo-go-driver#2312

https://github.com/mongodb/mongo-go-driver/releases/tag/v1.17.7
🐛 Fixed
GODRIVER-3770 Fix buffer handling in GSSAPI error description and username functions in mongodb/mongo-go-driver#2291
📝 Other Changes
Add more visible deprecation banner to the 1.17 readme in mongodb/mongo-go-driver#2233
GODRIVER-3770 Remove libasan from gssapi tests in CI n mongodb/mongo-go-driver#2293
GODRIVER-3766 Remove deprecation notice for MergeClientOptions in mongodb/mongo-go-driver#2294

@matthiasbruns matthiasbruns requested a review from a team as a code owner March 10, 2026 07:33
@github-actions github-actions bot added kind/bugfix Bug kind/dependency dependency update, etc. size/xs Extra small labels Mar 10, 2026
go.mod Outdated
)

// until cosign pushes a release, we need to override its transient dependency on mongo-driver v1.17.6
replace go.mongodb.org/mongo-driver => go.mongodb.org/mongo-driver v1.17.6 // BDSA-2026-1849
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Doesn't 1.17.6 contain the vulnerability? We need to upgrade do we?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Additionally, we might not be affected by this vulnerability as its affecting CGO bindings for GSSAPI

The mongo-go-driver repository contains CGo bindings for GSSAPI (Kerberos) authentication on Linux and macOS.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

maybe we wait for the upstream pr merge

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You can also check the changelog of go.mongodb.org/mongo-driver from 1.17.6 to 1.17.9. If there are no breaking changes, it could be fine to pin it to 1.17.9.

We just need to make sure that (a) it works and (b) we will remove the pin if upstream catched up

On-behalf-of: SAP <matthias.bruns@sap.com>
Signed-off-by: Matthias Bruns <git@matthiasbruns.com>
)

// until cosign pushes a release, we need to override its transient dependency on mongo-driver v1.17.6
replace go.mongodb.org/mongo-driver => go.mongodb.org/mongo-driver v1.17.9 // BDSA-2026-1849
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

when do we ever use this driver? this looks like dead code. there is no mongo ever used right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

let's wait for upstream to provide a fix - I think I was a bit over ambitious

@matthiasbruns matthiasbruns deleted the deps/BDSA-2026-1849 branch March 10, 2026 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bugfix Bug kind/dependency dependency update, etc. size/xs Extra small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants