Skip to content

Conversation

@anik120
Copy link
Member

@anik120 anik120 commented Apr 24, 2025

Reverts #3550

go-jose/[email protected] is an indirect-indirect dependency, and does not need to be
replace-ed since it's absent to begin with.

Copy link
Contributor

@tmshort tmshort left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Apr 24, 2025

replace (
github.com/go-jose/go-jose/v4 => github.com/go-jose/go-jose/v4 v4.0.5 // CVE-2025-27144
// controller runtime
Copy link
Contributor

Choose a reason for hiding this comment

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

We can just bump in the indirect call.
It has no need for replaces

Copy link
Contributor

Choose a reason for hiding this comment

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

That was the point... but in this case there is no reference to it anywhere, so we are removing it.

@tmshort tmshort enabled auto-merge April 24, 2025 15:06
@tmshort
Copy link
Contributor

tmshort commented Apr 24, 2025

/retest-required

@tmshort
Copy link
Contributor

tmshort commented Apr 24, 2025

/approve

@tmshort tmshort added this pull request to the merge queue Apr 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 24, 2025
@tmshort tmshort added this pull request to the merge queue Apr 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 24, 2025
@tmshort tmshort added this pull request to the merge queue Apr 25, 2025
Merged via the queue into master with commit c1b355d Apr 25, 2025
25 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants