Skip to content
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.

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

adds custom challenge for first method overload of confirmSignIn #2316

Merged
merged 1 commit into from
Dec 30, 2020

Conversation

haverchuck
Copy link
Contributor

Issue #, if available:
aws-amplify/amplify-flutter#271
aws-amplify/amplify-android#805
#1761

Description of changes:
If confirmSignIn is called without a map for custom challenge, we simply assume that the signInChallengeResponse variable as the attribute for the "ANSWER" key in the .setChallengeResponse's map parameter.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@haverchuck
Copy link
Contributor Author

The fix here is also similar to what amplify-js does.

@haverchuck haverchuck marked this pull request as ready for review December 30, 2020 21:21
@jamesonwilliams jamesonwilliams merged commit 1383637 into aws-amplify:main Dec 30, 2020
hannesj added a commit to hannesj/amplify-flutter that referenced this pull request Jan 26, 2021
Update aws-auth-cognito version for Android. This fixes aws-amplify#271 and aws-amplify#276.
The the custom challenge answer was fixed in aws-amplify/aws-sdk-android#2316. It was released in version v2.21.0 of the AWS Android SDK, which was included in the version 1.6.9 of the Amplify Android SDK.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants