fix(datastore): Adding secureConnectionFailed as a retryable error (#…

…3475)
aws-amplify · Jan 18, 2024 · 3a47dd6 · 3a47dd6
1 parent 417efad
commit 3a47dd6
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/AmplifyPlugins/DataStore/Sources/AWSDataStorePlugin/Sync/RequestRetryablePolicy.swift b/AmplifyPlugins/DataStore/Sources/AWSDataStorePlugin/Sync/RequestRetryablePolicy.swift
@@ -41,6 +41,7 @@ class RequestRetryablePolicy: RequestRetryable {
              .dataNotAllowed,
              .cannotParseResponse,
              .networkConnectionLost,
+             .secureConnectionFailed,
              .userAuthenticationRequired:
             let waitMillis = retryDelayInMillseconds(for: attemptNumber)
             return RequestRetryAdvice(shouldRetry: true, retryInterval: .milliseconds(waitMillis))

diff --git a/...ifyPlugins/DataStore/Tests/AWSDataStorePluginTests/Sync/RequestRetryablePolicyTests.swift b/...ifyPlugins/DataStore/Tests/AWSDataStorePluginTests/Sync/RequestRetryablePolicyTests.swift
@@ -210,6 +210,17 @@ class RequestRetryablePolicyTests: XCTestCase {
         XCTAssertEqual(retryAdvice.retryInterval, defaultTimeout)
     }
 
+    func testSecureConnectionFailedError() {
+        let retryableErrorCode = URLError.init(.secureConnectionFailed)
+
+        let retryAdvice = retryPolicy.retryRequestAdvice(urlError: retryableErrorCode,
+                                                         httpURLResponse: nil,
+                                                         attemptNumber: 1)
+
+        XCTAssert(retryAdvice.shouldRetry)
+        assertMilliseconds(retryAdvice.retryInterval, greaterThan: 200, lessThan: 300)
+    }
+
     func testMaxValueRetryDelay() {
         let retryableErrorCode = URLError.init(.timedOut)