Skip to content

Commit f47d335

Browse files
authored
fix(Auth): Fixing handling of service SdkError in Auth tasks (#2859)
1 parent a887257 commit f47d335

26 files changed

+92
-44
lines changed

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/AWSAuthChangePasswordTask.swift

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,10 @@ class AWSAuthChangePasswordTask: AuthChangePasswordTask, DefaultLogger {
3939
let accessToken = try await taskHelper.getAccessToken()
4040
try await changePassword(with: accessToken)
4141
log.verbose("Received success")
42-
} catch let error as ChangePasswordOutputError {
42+
} catch let error as AuthErrorConvertible {
4343
throw error.authError
44+
} catch let error as AuthError {
45+
throw error
4446
} catch let error {
4547
throw AuthError.configuration("Unable to execute auth task", AuthPluginErrorConstants.configurationError, error)
4648
}

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/AWSAuthConfirmResetPasswordTask.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,7 @@ class AWSAuthConfirmResetPasswordTask: AuthConfirmResetPasswordTask, DefaultLogg
3434
do {
3535

3636
try await confirmResetPassword()
37-
} catch let error as ConfirmForgotPasswordOutputError {
38-
throw error.authError
39-
} catch let error as SdkError<ConfirmForgotPasswordOutputError> {
37+
} catch let error as AuthErrorConvertible {
4038
throw error.authError
4139
} catch let error as AuthError {
4240
throw error

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/AWSAuthResendSignUpCodeTask.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,7 @@ class AWSAuthResendSignUpCodeTask: AuthResendSignUpCodeTask, DefaultLogger {
3636
let details = try await resendSignUpCode()
3737
log.verbose("Received result")
3838
return details
39-
} catch let error as ResendConfirmationCodeOutputError {
40-
throw error.authError
41-
} catch let error as SdkError<ResendConfirmationCodeOutputError> {
39+
} catch let error as AuthErrorConvertible {
4240
throw error.authError
4341
} catch let error as AuthError {
4442
throw error

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/AWSAuthResetPasswordTask.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,7 @@ class AWSAuthResetPasswordTask: AuthResetPasswordTask, DefaultLogger {
3636
let result = try await resetPassword()
3737
log.verbose("Received result")
3838
return result
39-
} catch let error as ForgotPasswordOutputError {
40-
throw error.authError
41-
} catch let error as SdkError<ForgotPasswordOutputError> {
39+
} catch let error as AuthErrorConvertible {
4240
throw error.authError
4341
} catch let error as AuthError {
4442
throw error

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/DeviceTasks/AWSAuthFetchDevicesTask.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,7 @@ class AWSAuthFetchDevicesTask: AuthFetchDevicesTask {
3636
let accessToken = try await taskHelper.getAccessToken()
3737
let devices = try await fetchDevices(with: accessToken)
3838
return devices
39-
} catch let error as ListDevicesOutputError {
40-
throw error.authError
41-
} catch let error as SdkError<ListDevicesOutputError> {
39+
} catch let error as AuthErrorConvertible {
4240
throw error.authError
4341
} catch let error as AuthError {
4442
throw error

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/DeviceTasks/AWSAuthForgetDeviceTask.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,7 @@ class AWSAuthForgetDeviceTask: AuthForgetDeviceTask {
3838
let accessToken = try await taskHelper.getAccessToken()
3939
let username = try await getCurrentUsername()
4040
try await forgetDevice(with: accessToken, username: username)
41-
} catch let error as ForgetDeviceOutputError {
42-
throw error.authError
43-
} catch let error as SdkError<ForgetDeviceOutputError> {
41+
} catch let error as AuthErrorConvertible {
4442
throw error.authError
4543
} catch let error as AuthError {
4644
throw error

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/DeviceTasks/AWSAuthRememberDeviceTask.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,7 @@ class AWSAuthRememberDeviceTask: AuthRememberDeviceTask {
3737
let accessToken = try await taskHelper.getAccessToken()
3838
let username = try await getCurrentUsername()
3939
try await rememberDevice(with: accessToken, username: username)
40-
} catch let error as UpdateDeviceStatusOutputError {
41-
throw error.authError
42-
} catch let error as SdkError<UpdateDeviceStatusOutputError> {
40+
} catch let error as AuthErrorConvertible {
4341
throw error.authError
4442
} catch let error as AuthError {
4543
throw error

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/UserTasks/AWSAuthAttributeResendConfirmationCodeTask.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ class AWSAuthAttributeResendConfirmationCodeTask: AuthAttributeResendConfirmatio
3535
let accessToken = try await taskHelper.getAccessToken()
3636
let devices = try await initiateGettingVerificationCode(with: accessToken)
3737
return devices
38-
} catch let error as GetUserAttributeVerificationCodeOutputError {
38+
} catch let error as AuthErrorConvertible {
3939
throw error.authError
4040
} catch let error as AuthError {
4141
throw error

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/UserTasks/AWSAuthConfirmUserAttributeTask.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ class AWSAuthConfirmUserAttributeTask: AuthConfirmUserAttributeTask {
3535
await taskHelper.didStateMachineConfigured()
3636
let accessToken = try await taskHelper.getAccessToken()
3737
try await confirmUserAttribute(with: accessToken)
38-
} catch let error as VerifyUserAttributeOutputError {
38+
} catch let error as AuthErrorConvertible {
3939
throw error.authError
4040
} catch let error as AuthError {
4141
throw error

AmplifyPlugins/Auth/Sources/AWSCognitoAuthPlugin/Task/UserTasks/AWSAuthFetchUserAttributeTask.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ class AWSAuthFetchUserAttributeTask: AuthFetchUserAttributeTask {
3535
await taskHelper.didStateMachineConfigured()
3636
let accessToken = try await taskHelper.getAccessToken()
3737
return try await getUserAttributes(with: accessToken)
38-
} catch let error as GetUserOutputError {
38+
} catch let error as AuthErrorConvertible {
3939
throw error.authError
4040
} catch let error as AuthError {
4141
throw error

0 commit comments

Comments
 (0)