Polly.Extensions.Http icon indicating copy to clipboard operation
Polly.Extensions.Http copied to clipboard

500 status code should not be considered as transient error

Open BDavidRamesh opened this issue 3 years ago • 1 comments

Current Problem Statement: the current 5XX check includes 500 which is not TransientError, 500 error requires code changes or query changes by the dev team.

  1. so retry for 500 always returns 500
  2. if there is dependency on Service-1 for 3 different Api calls for 3 different workflows, opening circuit for 500 error on Api-1 makes the entire Service unavailable.

Proposed Fix: response status code should be ">500" instead of ">=500"

BDavidRamesh avatar Jun 15 '22 16:06 BDavidRamesh

Any discussion around this a little bit controversial topic? :)

savbace avatar Mar 15 '23 18:03 savbace