Skip to content

Commit

Permalink
Remove TransferChunks gRPC method (#6543)
Browse files Browse the repository at this point in the history
Signed-off-by: SungJin1212 <[email protected]>
  • Loading branch information
SungJin1212 authored Jan 24, 2025
1 parent 69f5fb4 commit a01bfa1
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions pkg/cortex/cortex.go
Original file line number Diff line number Diff line change
Expand Up @@ -344,13 +344,10 @@ func New(cfg Config) (*Cortex, error) {
tenant.WithDefaultResolver(tenant.NewMultiResolver())
}

// Don't check auth header on TransferChunks, as we weren't originally
// sending it and this could cause transfers to fail on update.
cfg.API.HTTPAuthMiddleware = fakeauth.SetupAuthMiddleware(&cfg.Server, cfg.AuthEnabled,
// Also don't check auth for these gRPC methods, since single call is used for multiple users (or no user like health check).
[]string{
"/grpc.health.v1.Health/Check",
"/cortex.Ingester/TransferChunks",
"/frontend.Frontend/Process",
"/frontend.Frontend/NotifyClientShutdown",
"/schedulerpb.SchedulerForFrontend/FrontendLoop",
Expand Down

0 comments on commit a01bfa1

Please sign in to comment.