Backport candidate: CVE-2025-13888 fix not present on v1.13 branch
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 52/100
Research direction
First verify whether the v1.13 branch is still supported. If it is, inspect controllers/argocd_metrics_controller.go on v1.13 and compare it with commit bc6ac3e0 from PR #897; done means the namespace-isolation fix is present on v1.13, or an equivalent backport is provided. If the branch is EOL, document that status instead.
Written by the indexing model from the issue text.
Description
Hi maintainers,
While auditing supported branches for CVE backports, I noticed that the fix for CVE-2025-13888 (PR #897, commit bc6ac3e0, "fix CVE namespace-isolation break") does not appear to be present on the v1.13 branch.
What I checked
- The
v1.13branch HEAD does not contain the new symbols introduced by the fix:userDefinedMonitoringLabelconstant — not present incontrollers/argocd_metrics_controller.goonv1.13openshift.io/user-monitoringlabel literal — not presentstrings.HasPrefix(namespace.Name, "openshift-")guard — not present
- The pre-fix code path (
namespace.Labels[clusterMonitoringLabel] = "true"unconditional) is still present. git compare v1.13...bc6ac3e0reports the branch is 16 commits behind the fix commit and the fix is not in the merged set.
Why this matters
Without the openshift- prefix guard, the operator may add the cluster-monitoring label to namespaces it shouldn't, which is the namespace-isolation issue described in the advisory. If v1.13 is still a supported maintenance branch, a cherry-pick of bc6ac3e0 (or an equivalent fix) would close that gap.
If v1.13 is no longer supported / EOL, please feel free to close — happy to know either way.
Thanks for your work on this project.
— @vulgraph
- Dominant language
- Go
- Stars
- 188
- Forks
- 359
- Avg merge
- 4d 6h
- Merged PRs (30d)
- 27
Getting set up
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from redhat-developer/gitops-operator
-
Difficulty 1/5 Under an hour Newbie friendliness 86/100
redhat-developer/gitops-operator#1287 ·
Maintainers usually reply within 1 day
-
Difficulty 3/5 1-2 days Newbie friendliness 45/100
redhat-developer/gitops-operator#1311 ·
Maintainers usually reply within 1 day
-
Harden OpenShift manual-install SCC troubleshooting guidancePossibly taken @olivergondza claimed this 13 days ago. Open
redhat-developer/gitops-operator#1295 · 1 assignee ·
Maintainers usually reply within 1 day
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
redhat-developer/gitops-operator#634 · 1 comment ·
Maintainers usually reply within 1 day
-
triage:required
Difficulty 4/5 3-5 days Newbie friendliness 48/100
redhat-developer/gitops-operator#572 · 2 comments ·
Maintainers usually reply within 1 day
All issues in redhat-developer/gitops-operator
Similar issues
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 92/100
open-telemetry/opentelemetry-go-compile-instrumentation#1417 ·
Maintainers usually reply within 2 days
-
agent-research-finding agent-research-recommend chore ready-for-agent
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
jordansmall/spindrift#4068 · 1 comment ·
Maintainers usually reply within 1 day
-
Type/Task
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
OpenNSW/nsw-srilanka#537 ·
Maintainers usually reply within 1 day
-
security
Difficulty 2/5 1-2 days Newbie friendliness 62/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
Maintainers usually reply within 1 day