Skip to content

Commit 4df9905

Browse files
harsh62atierian
andauthored
chore: Update aws-swift-sdk to 0.5.0 (#2560)
* chore: Updating aws-swift-sdk to 0.4.0 * fix(auth): custom endpoint resolver (#2552) * chore: Updating aws-swift-sdk to 0.5.0 and fixing s3 configuration issues (#2556) * Update MockAWSClientConfiguration.swift * chore: Fixing build errors in Geo and Analytics (#2558) * chore: Fixing build errors in Geo and Analytics * Update Date+Formatting.swift * Update MockAWSClientConfiguration.swift * chore: Update error mapping for Auth to introduce ForbiddenException * fix: adding customized logic for creating presignedURL * fix: adding unknown default to error enums Co-authored-by: Ian Saultz <[email protected]>
1 parent 7228fde commit 4df9905

File tree

41 files changed

+322
-179
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

41 files changed

+322
-179
lines changed

AmplifyPlugins/Analytics/Sources/AWSPinpointAnalyticsPlugin/Support/Extensions/Date+Formatting.swift

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,14 @@ extension DateFormatter {
1515
formatter.locale = Locale(identifier: "en_US_POSIX")
1616
return formatter
1717
}()
18+
19+
static let iso8601DateFormatterWithFractionalSeconds: DateFormatter = {
20+
let formatter = DateFormatter()
21+
formatter.dateFormat = "yyyy-MM-dd'T'HH:mm:ss.SSSZZZZZ"
22+
formatter.timeZone = TimeZone(secondsFromGMT: 0)
23+
formatter.locale = Locale(identifier: "en_US_POSIX")
24+
return formatter
25+
}()
1826
}
1927

2028
extension Date {

AmplifyPlugins/Analytics/Sources/AWSPinpointAnalyticsPlugin/Support/Extensions/PinpointClient+CredentialsProvider.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,9 @@ import AWSPinpoint
1212
extension PinpointClient {
1313
convenience init(region: String, credentialsProvider: CredentialsProvider) throws {
1414
let configuration = try PinpointClientConfiguration(
15-
region: region,
1615
credentialsProvider: credentialsProvider,
17-
frameworkMetadata: AmplifyAWSServiceConfiguration.frameworkMetaData()
16+
frameworkMetadata: AmplifyAWSServiceConfiguration.frameworkMetaData(),
17+
region: region
1818
)
1919

2020
self.init(config: configuration)

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/AWSCognitoAuthPlugin+Configure.swift

Lines changed: 8 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -84,21 +84,12 @@ extension AWSCognitoAuthPlugin {
8484
private func makeUserPool() throws -> CognitoUserPoolBehavior {
8585
switch authConfiguration {
8686
case .userPools(let userPoolConfig), .userPoolsAndIdentityPools(let userPoolConfig, _):
87-
88-
let configuration: CognitoIdentityProviderClient.CognitoIdentityProviderClientConfiguration
89-
if let customEndpoint = userPoolConfig.endpoint {
90-
configuration = try .init(
91-
region: userPoolConfig.region,
92-
endpointResolver: customEndpoint.resolver,
93-
frameworkMetadata: AmplifyAWSServiceConfiguration.frameworkMetaData()
94-
)
95-
} else {
96-
configuration = try CognitoIdentityProviderClient.CognitoIdentityProviderClientConfiguration(
97-
region: userPoolConfig.region, frameworkMetadata: AmplifyAWSServiceConfiguration.frameworkMetaData())
98-
}
99-
87+
let configuration = try CognitoIdentityProviderClient.CognitoIdentityProviderClientConfiguration(
88+
endpointResolver: userPoolConfig.endpoint?.resolver,
89+
frameworkMetadata: AmplifyAWSServiceConfiguration.frameworkMetaData(),
90+
region: userPoolConfig.region
91+
)
10092
return CognitoIdentityProviderClient(config: configuration)
101-
10293
default:
10394
fatalError()
10495
}
@@ -108,7 +99,9 @@ extension AWSCognitoAuthPlugin {
10899
switch authConfiguration {
109100
case .identityPools(let identityPoolConfig), .userPoolsAndIdentityPools(_, let identityPoolConfig):
110101
let configuration = try CognitoIdentityClient.CognitoIdentityClientConfiguration(
111-
region: identityPoolConfig.region, frameworkMetadata: AmplifyAWSServiceConfiguration.frameworkMetaData())
102+
frameworkMetadata: AmplifyAWSServiceConfiguration.frameworkMetaData(),
103+
region: identityPoolConfig.region
104+
)
112105
return CognitoIdentityClient(config: configuration)
113106
default:
114107
fatalError()

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Service/ErrorMapping/ChangePasswordOutputError+AuthError.swift

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,11 @@ extension ChangePasswordOutputError: AuthErrorConvertible {
5555
return .service(limitExceededException.message ?? "Limit exceeded error",
5656
AuthPluginErrorConstants.limitExceededError,
5757
AWSCognitoAuthError.limitExceeded)
58+
case .forbiddenException(let forbiddenException):
59+
return .service(forbiddenException.message ?? "Access to the requested resource is forbidden",
60+
AuthPluginErrorConstants.forbiddenError)
61+
@unknown default:
62+
return .unknown("Unknown service error occurred")
5863
}
5964
}
6065
}

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Service/ErrorMapping/ConfirmForgotPasswordOutputError+AuthError.swift

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,6 +73,11 @@ extension ConfirmForgotPasswordOutputError: AuthErrorConvertible {
7373
let statusCode = serviceError._statusCode?.rawValue ?? -1
7474
let message = serviceError._message ?? ""
7575
return .unknown("Unknown service error occurred with status \(statusCode) \(message)")
76+
case .forbiddenException(let forbiddenException):
77+
return .service(forbiddenException.message ?? "Access to the requested resource is forbidden",
78+
AuthPluginErrorConstants.forbiddenError)
79+
@unknown default:
80+
return .unknown("Unknown service error occurred")
7681
}
7782
}
7883
}

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Service/ErrorMapping/ConfirmSignUpOutputError+AuthError.swift

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,12 @@ extension ConfirmSignUpOutputError: AuthErrorConvertible {
111111
let statusCode = serviceError._statusCode?.rawValue ?? -1
112112
let message = serviceError._message ?? ""
113113
return .unknown("Unknown service error occurred with status \(statusCode) \(message)")
114-
114+
115+
case .forbiddenException(let forbiddenException):
116+
return .service(forbiddenException.message ?? "Access to the requested resource is forbidden",
117+
AuthPluginErrorConstants.forbiddenError)
118+
@unknown default:
119+
return .unknown("Unknown service error occurred")
115120
}
116121
}
117122

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Service/ErrorMapping/DeleteUserOutputError+AuthError.swift

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,12 @@ extension DeleteUserOutputError: AuthErrorConvertible {
4646
let statusCode = serviceError._statusCode?.rawValue ?? -1
4747
let message = serviceError._message ?? ""
4848
return .unknown("Unknown service error occurred with status \(statusCode) \(message)")
49+
50+
case .forbiddenException(let forbiddenException):
51+
return .service(forbiddenException.message ?? "Access to the requested resource is forbidden",
52+
AuthPluginErrorConstants.forbiddenError)
53+
@unknown default:
54+
return .unknown("Unknown service error occurred")
4955
}
5056
}
5157
}

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Service/ErrorMapping/ForgetDeviceOutputError+AuthError.swift

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,12 @@ extension ForgetDeviceOutputError: AuthErrorConvertible {
4949
let statusCode = serviceError._statusCode?.rawValue ?? -1
5050
let message = serviceError._message ?? ""
5151
return .unknown("Unknown service error occurred with status \(statusCode) \(message)")
52+
53+
case .forbiddenException(let forbiddenException):
54+
return .service(forbiddenException.message ?? "Access to the requested resource is forbidden",
55+
AuthPluginErrorConstants.forbiddenError)
56+
@unknown default:
57+
return .unknown("Unknown service error occurred")
5258
}
5359
}
5460
}

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Service/ErrorMapping/ForgotPasswordOutputError+AuthError.swift

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,12 @@ extension ForgotPasswordOutputError: AuthErrorConvertible {
8181
let statusCode = serviceError._statusCode?.rawValue ?? -1
8282
let message = serviceError._message ?? ""
8383
return .unknown("Unknown service error occurred with status \(statusCode) \(message)")
84+
85+
case .forbiddenException(let forbiddenException):
86+
return .service(forbiddenException.message ?? "Access to the requested resource is forbidden",
87+
AuthPluginErrorConstants.forbiddenError)
88+
@unknown default:
89+
return .unknown("Unknown service error occurred")
8490
}
8591
}
8692
}

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Service/ErrorMapping/GetCredentialsForIdentityOutputError+AuthError.swift

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,8 @@ extension GetCredentialsForIdentityOutputError: AuthErrorConvertible {
4545
let statusCode = unknownAWSHttpServiceError._statusCode?.rawValue ?? -1
4646
let message = unknownAWSHttpServiceError._message ?? ""
4747
return .unknown("Unknown service error occurred with status \(statusCode) \(message)")
48+
@unknown default:
49+
return .unknown("Unknown service error occurred")
4850
}
4951
}
5052
}

0 commit comments

Comments
 (0)