From c04ce88d9e7e8376ddcc90a941fff1853801932c Mon Sep 17 00:00:00 2001 From: Abhishek Pandey Date: Wed, 17 May 2023 15:06:11 -0700 Subject: [PATCH] Remove 503, 504 retries from corso retry handler (#3435) Remove overlap between corso retry handler & kiota retry handler. Since kiota retry handler supports [503, 504 retries](https://github.com/microsoft/kiota-http-go/blob/fb98fba2da6a1e673c3f111817387aac6184c67a/retry_handler.go#L170), we don't need to duplicate these retries in corso retry handler. --- #### 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 - [x] :bug: Bugfix - [ ] :world_map: Documentation - [ ] :robot: Supportability/Tests - [ ] :computer: CI/Deployment - [ ] :broom: Tech Debt/Cleanup #### Issue(s) * # #### Test Plan - [x] :muscle: Manual - [ ] :zap: Unit test - [ ] :green_heart: E2E --- src/internal/connector/graph/middleware.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/internal/connector/graph/middleware.go b/src/internal/connector/graph/middleware.go index d78350c4f..1a3933460 100644 --- a/src/internal/connector/graph/middleware.go +++ b/src/internal/connector/graph/middleware.go @@ -343,9 +343,7 @@ func (mw RetryMiddleware) retryRequest( var retryableRespCodes = []int{ http.StatusInternalServerError, - http.StatusServiceUnavailable, http.StatusBadGateway, - http.StatusGatewayTimeout, } func (mw RetryMiddleware) isRetriableRespCode(ctx context.Context, resp *http.Response, code int) bool {