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

Clear CustomAuthChallenge Task on SignOut #2265

Merged
merged 2 commits into from
Jan 31, 2020

Conversation

kneekey23
Copy link
Contributor

Issue #, if available: Providing a fix for #2261

Description of changes:
CustomAuthChallengeTask does not clear on sign out so if sign in challenge wasn't completed and sign out was called first, you receive an error of "Missing required parameter ANSWER"

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

@kneekey23 kneekey23 self-assigned this Jan 30, 2020
@kneekey23 kneekey23 added mobile client Issues related to AWSMobileClient Pull Request labels Jan 30, 2020
@kneekey23 kneekey23 changed the base branch from master to develop January 30, 2020 21:48
@wooj2
Copy link
Contributor

wooj2 commented Jan 30, 2020

  1. Commits for this package should be against develop
  2. please update change log
  3. Code looks fine, but... maybe Jithin could take a look, as he has modified custom auth more than I have :)

@kneekey23 kneekey23 requested a review from royjit January 30, 2020 21:49
Copy link
Member

@palpatim palpatim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, assuming @royjit has no objections

@royjit
Copy link
Member

royjit commented Jan 31, 2020

LGTM

@kneekey23 kneekey23 merged commit b51a064 into develop Jan 31, 2020
thomers pushed a commit to thomers/aws-sdk-ios that referenced this pull request Feb 13, 2020
…2.12.7-Upgrade

* commit '0ce485952faf24fbe6e9fb8cd064a9e844d95bad': (187 commits)
  [bump version 2.12.7] (aws-amplify#2286)
  Copy x86_64-apple-ios-simulator swiftmodule files for AWSMobileClient… (aws-amplify#2281)
  CHANGELOG for 2020-02-10-08-01 model updates (aws-amplify#2285)
  [iot] Update models to latest (aws-amplify#2284)
  [ec2] Update models to latest (aws-amplify#2283)
  Update CHANGELOG (aws-amplify#2279)
  fix CocoaPods publish deploy logic (aws-amplify#2277)
  Allow for static lib generation using workaround for swift/obj-c mixed projects (aws-amplify#2267)
  EC2 integration test bug fix (aws-amplify#2270)
  Remove all user attributes for new password required flow (aws-amplify#2203) (aws-amplify#2240) (aws-amplify#2255)
  Fix NSInternalInconsistencyException on new password required flow (aws-amplify#2254)
  Clear CustomAuthChallenge Task on SignOut (aws-amplify#2265)
  CHANGELOG for 2020-01-26-08-15 model updates (aws-amplify#2259)
  [lambda] Update models to latest (aws-amplify#2258)
  [kms] Update models to latest (aws-amplify#2257)
  [ec2] Update models to latest (aws-amplify#2256)
  Updating CHANGELOG per request in aws-amplify#2240 (aws-amplify#2252)
  Filter out non-mutable attribute "email_verified" for new password required flow (aws-amplify#2203) (aws-amplify#2240)
  CHANGELOG for 2020-01-19-08-15 model updates (aws-amplify#2244)
  [ec2] Update models to latest (aws-amplify#2243)
  ...

# Conflicts:
#	AWSAuthSDK/Sources/AWSAuthUI/AWSAuthUIViewController.m
#	AWSAuthSDK/Sources/AWSAuthUI/AWSSignInViewController.m
#	AWSAuthSDK/Sources/AWSAuthUI/SignIn.storyboard
#	AWSAuthSDK/Sources/AWSMobileClient/AWSMobileOptions.swift
#	AWSAuthSDK/Sources/AWSMobileClient/Internal/_AWSMobileClient.m
#	AWSAuthSDK/Sources/AWSUserPoolsSignIn/UserPoolsUI/AWSUserPools.storyboard
#	AWSAuthSDK/Sources/AWSUserPoolsSignIn/UserPoolsUI/AWSUserPoolsUIHelper.m
@palpatim palpatim deleted the kneekey/mobileclient-customauth-bug branch April 29, 2020 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mobile client Issues related to AWSMobileClient
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants