Add support for default max tries setting for Http requests #282
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WHY are these changes introduced?
Fixes #281
Shopify\Clients\Http
is able to retry a request if response code is retriable andtries
parameter given to request method. This change gives ability to set a default max tries count for each Http request unless overridden by request method parameter. This way we can also use retries with REST resources.WHAT is this pull request doing?
Adds a
$maxTries
parameter toShopify\Context::initialize
method with a default value of 1. Assigns it toShopify\Context::$MAX_TRIES
and uses it as default value inrequest
method ofShopify\Client\Http
.Type of change
Checklist