Skip to content

chore(deps): bump google.golang.org/grpc to v1.82.1 - #835

Merged
VaniHaripriya merged 2 commits into
red-hat-data-services:rhoai-3.6-ea.2from
sduvvuri1603:fix/grpc-1.82.1-rhoai-3.6-ea.2
Sep 17, 2026
Merged

VaniHaripriya merged 2 commits into
red-hat-data-services:rhoai-3.6-ea.2from
sduvvuri1603:fix/grpc-1.82.1-rhoai-3.6-ea.2

Conversation

@sduvvuri1603

@sduvvuri1603 sduvvuri1603 commented Sep 17, 2026

Copy link
Copy Markdown

Fixes RHOAIENG-94998, RHOAIENG-94999

Summary

  • Bump google.golang.org/grpc from v1.71.1 to v1.82.1 so both images (argoexec and workflowcontroller) pick up GHSA-p77j-4mvh-x3m3 and GHSA-hrxh-6v49-42gf.
  • Scanner "fixed in 1.79.3" only covers the path-authz CVE; GHSA-hrxh needs 1.82.1. Associated protobuf/genproto/otel/golang.org/x/* modules moved with MVS.
  • Follow-up in workflow/artifacts/gcs/gcs.go: the oauth2 bump made staticcheck SA1019 fail on google.CredentialsFromJSON. That call is not part of the GHSAs; switch it to option.WithCredentialsJSON so Lint is not a false CVE signal.

Test plan

  • go build of workflow-controller, argoexec, and argo
  • go test ./util/grpc/ ./server/auth/
  • Lint (SA1019 in gcs.go)

@sduvvuri1603
sduvvuri1603 force-pushed the fix/grpc-1.82.1-rhoai-3.6-ea.2 branch 2 times, most recently from b70e471 to 7ae4d79 Compare September 17, 2026 15:03
Fixes GHSA-p77j-4mvh-x3m3 and GHSA-hrxh-6v49-42gf on rhoai-3.6-ea.2
(RHOAIENG-94998, RHOAIENG-94999).

Signed-off-by: sduvvuri1603 <sduvvuri@redhat.com>
staticcheck SA1019 started failing after the grpc/oauth2 bump. Pass
the service account JSON through option.WithCredentialsJSON instead.

Signed-off-by: sduvvuri1603 <sduvvuri@redhat.com>
@sduvvuri1603
sduvvuri1603 force-pushed the fix/grpc-1.82.1-rhoai-3.6-ea.2 branch from ed370aa to 1c4bf89 Compare September 17, 2026 15:46
@VaniHaripriya

Copy link
Copy Markdown

/lgtm

@VaniHaripriya
VaniHaripriya merged commit bc397a3 into red-hat-data-services:rhoai-3.6-ea.2 Sep 17, 2026
10 of 27 checks passed
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.

2 participants