You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by "Thierry Ygé (Jira)" <ji...@apache.org> on 2022/02/05 15:47:00 UTC
[jira] [Created] (SLING-11125) SlingClient.exists(...) should use expected status in order to support http retries parameters
Thierry Ygé created SLING-11125:
-----------------------------------
Summary: SlingClient.exists(...) should use expected status in order to support http retries parameters
Key: SLING-11125
URL: https://issues.apache.org/jira/browse/SLING-11125
Project: Sling
Issue Type: Bug
Components: Apache Sling Testing Clients
Affects Versions: Apache Sling Testing Clients 3.0.8
Reporter: Thierry Ygé
Currently if tests need to retries on some errors which are likely to happen in clustered setup (AEM in Cloud for example).
usually the use of "-Dsling.it.http.retriesErrorCodes=401,404,405,429,500,501,503" would include 404 cases.
The issue is then SlingClient.exists(...) would always fail in case we try to check for "not existing path" as the test would expect also 404 to be returned.
The current implementation of ServerErrorRetryStrategy support that if the expected values are passed , this would bypass the retriesErrorCodes.
Thus it would make sense to pass the possible returned code for the request done by SlingClient.exists(...) call.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)