Skip to content

Prevent crash due to nil result when Cognito responds with error #8

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
andhieka wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Prevent crash due to nil result when Cognito responds with error #8

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions flutter_aws_amplify_cognito/ios/Classes/SwiftFlutterAwsAmplifyCognito.swift
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error initializing AWSMobileClient", details: error?.localizedDescription))
}
return
}
switch (userState) {
case .guest:
Expand Down Expand Up @@ -81,6 +82,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error signing out", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(true)
Expand All @@ -94,6 +96,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error getting user attributes", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(userAttributes)
Expand All @@ -115,6 +118,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error getting tokens", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(
Expand All @@ -132,6 +136,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error getting idToken", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(tokens?.idToken?.tokenString)
Expand All @@ -145,6 +150,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error getting accessToken", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(tokens?.accessToken?.tokenString)
Expand All @@ -158,6 +164,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error getting refreshToken", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(tokens?.refreshToken?.tokenString)
Expand All @@ -171,6 +178,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error getting AWS credentails", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result([
Expand All @@ -190,6 +198,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error signing up", details: error?.localizedDescription))
}
return
}
if (signUpResult!.signUpConfirmationState == SignUpConfirmationState.confirmed) {
DispatchQueue.main.async {
Expand Down Expand Up @@ -220,6 +229,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error confirming sign up", details: error?.localizedDescription))
}
return
}
if (signUpResult!.signUpConfirmationState == SignUpConfirmationState.confirmed) {
DispatchQueue.main.async {
Expand Down Expand Up @@ -250,6 +260,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error resending signing up code", details: error?.localizedDescription))
}
return
}
let userCodeDeliveryDetails = signUpResult?.codeDeliveryDetails
DispatchQueue.main.async {
Expand All @@ -269,6 +280,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error signing in", details: error?.localizedDescription))
}
return
}
var signInState: String = ""

Expand Down Expand Up @@ -314,6 +326,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error confirming sign in", details: error?.localizedDescription))
}
return
}
var signInState: String = ""

Expand Down Expand Up @@ -359,6 +372,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error requesting password reset", details: error?.localizedDescription))
}
return
}

var forgotPasswordState: String = ""
Expand Down Expand Up @@ -394,6 +408,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error confirming password reset", details: error?.localizedDescription))
}
return
}

var forgotPasswordState: String = ""
Expand Down Expand Up @@ -427,6 +442,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error tracking device", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(true)
Expand All @@ -441,6 +457,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error untracking device", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(true)
Expand All @@ -454,6 +471,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error forgetting device", details: error?.localizedDescription))
}
return
}
DispatchQueue.main.async {
result(true)
Expand All @@ -467,6 +485,7 @@ class SwiftFlutterAwsAmplifyCognito {
DispatchQueue.main.async {
result(FlutterError(code: "Error", message: "Error getting device details", details: error?.localizedDescription))
}
return
}
let formatter = DateFormatter()
formatter.dateFormat = "yyyy-MM-dd HH:mm:ss ZZZ"
Expand Down