<#6997 Add IdentityService (UserInfo) stub to runs...
# flyte-github
a
#6997 Add IdentityService (UserInfo) stub to runs service Pull request opened by pingsutw Tracking issue N/A Why are the changes needed? Add a dummy service first, will wire up with real auth in the follow-up PRs The
IdentityService
with its
UserInfo
RPC exists in the proto/generated code (
flyteidl2/auth/identity.proto
) but has no handler implementation in the v2 codebase. This adds a stub so the endpoint is mounted and available for future auth wiring. What changes were proposed in this pull request? • Added
runs/service/identity_service.go
— implements
authconnect.IdentityServiceHandler
with a stub
UserInfo
method returning an empty
UserInfoResponse
• Updated
runs/setup.go
— imports
authconnect
and mounts the
IdentityService
handler on the mux • Added
runs/service/identity_service_test.go
— unit test confirming
UserInfo
returns a valid empty response How was this patch tested? •
go build ./runs/...
compiles successfully •
go test ./runs/service/ -run TestIdentityService
passes • Unit test verifies
UserInfo
returns a non-nil empty response without error Labels • added: New IdentityService handler stub Setup process N/A Screenshots N/A Check all the applicable boxes • I updated the documentation accordingly. • All new and existing tests passed. • All commits are signed-off. •
main
• #6583 • #6997 👈 flyteorg/flyte