chore(deps): bump github.com/golang-jwt/jwt/v4 to v4.5.2 (CVE-2025-30204) - #274
chore(deps): bump github.com/golang-jwt/jwt/v4 to v4.5.2 (CVE-2025-30204)#274shahsahil264 wants to merge 1 commit into
Conversation
) Addresses CVE-2025-30204 (GHSA-mh63-6h87-95cp / GO-2025-3553): ParseUnverified used strings.Split on untrusted input, allowing excessive memory allocation via a crafted Authorization header with many period characters. golang-jwt is an indirect/transitive dependency only; cincinnati-operator does not call ParseUnverified or any golang-jwt API directly. govulncheck confirms no reachable call path to the vulnerable function before or after this bump. Signed-off-by: Sahil Shah <sahshah@redhat.com>
|
Pipeline controller notification For optional jobs, comment This repository is configured in: LGTM mode |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: openshift/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: shahsahil264 The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Addresses CVE-2025-30204 (GHSA-mh63-6h87-95cp / GO-2025-3553).
golang-jwt/jwt/v4is an indirect/transitive dependency of this repo.Parser.ParseUnverified(the vulnerable function, which usesstrings.Spliton untrusted input) is never called bycincinnati-operator's own code — confirmed via
govulncheck, whichplaces this CVE under "imported but not called" both before and after
this bump.
This bump is pre-emptive dependency hygiene; the corresponding Jira
ticket (OCPBUGS-105272) will be closed as Not-A-Bug since no reachable
exploitation path exists in this component.
Verified: go build, go vet, go test ./controllers/... all pass.
govulncheck output attached in the Jira ticket comment.