From 60d173b25b28c600236f8919b8f709bbe7328175 Mon Sep 17 00:00:00 2001 From: ashmrtn Date: Fri, 17 Mar 2023 17:14:03 -0700 Subject: [PATCH] Log retry after for throttled requests (#2847) This seems to be the header that is used to communicate the delay in seconds to wait. --- #### Does this PR need a docs update or release note? - [ ] :white_check_mark: Yes, it's included - [ ] :clock1: Yes, but in a later PR - [x] :no_entry: No #### Type of change - [ ] :sunflower: Feature - [ ] :bug: Bugfix - [ ] :world_map: Documentation - [ ] :robot: Test - [ ] :computer: CI/Deployment - [x] :broom: Tech Debt/Cleanup #### Issue(s) * closes #2846 #### Test Plan - [x] :muscle: Manual - [ ] :zap: Unit test - [ ] :green_heart: E2E --- src/internal/connector/graph/service.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/internal/connector/graph/service.go b/src/internal/connector/graph/service.go index ebeb5a408..69403fb19 100644 --- a/src/internal/connector/graph/service.go +++ b/src/internal/connector/graph/service.go @@ -303,7 +303,6 @@ func (handler *LoggingMiddleware) Intercept( // with a slimmer reference for telemetry/supportability purposes. if logger.DebugAPI || os.Getenv(logGraphRequestsEnvKey) != "" { log.Errorw("non-2xx graph api response", "response", getRespDump(ctx, resp, true)) - return resp, err } @@ -314,7 +313,8 @@ func (handler *LoggingMiddleware) Intercept( log.With( "limit", resp.Header.Get(rateLimitHeader), "remaining", resp.Header.Get(rateRemainingHeader), - "reset", resp.Header.Get(rateResetHeader)) + "reset", resp.Header.Get(rateResetHeader), + "retry-after", resp.Header.Get(retryAfterHeader)) } else if resp.StatusCode/100 == 4 { log.With("response", getRespDump(ctx, resp, true)) }