Skip to content
This repository has been archived by the owner on Nov 13, 2023. It is now read-only.

Update HttpPolicyExtensions.cs #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BDavidRamesh
Copy link

@BDavidRamesh BDavidRamesh commented Jun 15, 2022

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 1 Service for 2 different Api calls for 2 different workflows, opening circuit makes the entire Service unavailable even in case of 500 error.

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant