Removed maximum retry limit in the case of HTTP 404 to allow constant polling of upstream services in the case of an HTTP 404.
Updated logging level from error to warn
Added unit test to cover Observable error scenarios
Motivation and Context
We want to allow remote 404s to continue polling until a connection can be established. There are use cases where this is a valid scenario and prevents the need for application restarts
How Has This Been Tested?
This change was tested in tandem with changes to @creditkarma/dynamic-config internally and validated to be working as expected
Types of changes
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes (existing integration test covers the retry scenario).
Description
Motivation and Context
We want to allow remote 404s to continue polling until a connection can be established. There are use cases where this is a valid scenario and prevents the need for application restarts
How Has This Been Tested?
This change was tested in tandem with changes to @creditkarma/dynamic-config internally and validated to be working as expected
Types of changes
Checklist: