Skip to content

Restore google.golang.org/grpc to v1.79.3 (re-fix CVE)#437

Merged
yingzhan-msft merged 1 commit into
mainfrom
users/yingzhan/restore-grpc
Jun 8, 2026
Merged

Restore google.golang.org/grpc to v1.79.3 (re-fix CVE)#437
yingzhan-msft merged 1 commit into
mainfrom
users/yingzhan/restore-grpc

Conversation

@yingzhan-msft

Copy link
Copy Markdown
Contributor

Summary

Commit 944d459 ("Drop dead Error field from *IPUpdateResponse", merged in #435) regenerated .pb.go files via make generate inside the proto-gen container. That container had a stale module graph and silently rewrote go.mod to drop google.golang.org/grpc from v1.79.3 → v1.59.0.

v1.79.3 was specifically pinned in commit 5a3c9db (#429) to fix a gRPC authorization-bypass CVE — so the unintended downgrade re-introduced the vulnerability on main.

Changes

  • go.mod / go.sum: restore google.golang.org/grpc v1.79.3 and refresh transitive deps.
  • No source-code changes.

Testing

  • go build ./...
  • go test ./...

Commit 944d459 ("Drop dead Error field from *IPUpdateResponse")
regenerated .pb.go files via `make generate` inside the proto-gen
container, which had a stale module graph and silently rewrote
go.mod to drop grpc from v1.79.3 back to v1.59.0. v1.79.3 was
specifically pinned by commit 5a3c9db to fix a gRPC authorization
bypass CVE, so this undo re-introduced the vulnerability.

Restore the intended version. No code changes — only go.mod / go.sum.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@yingzhan-msft

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@yingzhan-msft yingzhan-msft merged commit a7feca9 into main Jun 8, 2026
7 checks passed
yingzhan-msft added a commit to microsoft/moc-sdk-for-go that referenced this pull request Jun 8, 2026
Picks up microsoft/moc#437 which restores google.golang.org/grpc
to v1.79.3 (CVE fix that was accidentally undone in v0.43.0).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

3 participants