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

AuthUI ForceChangePasswordView Form Name Mismatch #586

Closed
dennisunderwood opened this issue Nov 21, 2018 · 3 comments
Closed

AuthUI ForceChangePasswordView Form Name Mismatch #586

dennisunderwood opened this issue Nov 21, 2018 · 3 comments
Assignees
Labels
bug Something isn't working cognito Issues with the AWS Android SDK for Cognito pending-community-response Issue is pending response from the issue requestor

Comments

@dennisunderwood
Copy link

Describe the bug
There is a mismatch between two variable names inside of the two associated files:
aws-sdk-android/aws-android-sdk-auth-userpools/src/main/java/com/amazonaws/mobile/auth/userpools/ForceChangePasswordView.java
and
aws-sdk-android/aws-android-sdk-auth-userpools/src/main/res/layout/activity_force_change_password.xml

The following mismatch is present, causing runtime crash:
activity_force_change_password.xml in line 37:
android:id="@+id/force_change_password_form"
ForceChangePasswordView.java in line 133:
forgotPassForm = (FormView) findViewById(R.id.forgot_password_form);

The second mismatch is the following:
activity_force_change_password.xml in line 50:
android:id="@+id/force_change_password_button"
ForceChangePasswordView.java in line 142:
confirmButton = (Button) findViewById(R.id.forgot_password_button);

To Reproduce
Steps to reproduce the behavior:

  1. Create new user in Cognito, or set state of user to FORCE_CHANGE_PASSWORD
  2. Login with temporary password, so that the AuthUI attempts to use the code path for force change password.
  3. Unhandled runtime crash of the app.

Which AWS service(s) are affected?
AWS Cognito - client side, not server
AWS AuthUI

Expected behavior
User should be redirected to the page asking for a new password to be set.

Environment(please complete the following information):

  • SDK Version: 2.7.6 - 8.0.3

Device Information (please complete the following information):

  • Device: Samsung Galaxy S7
  • Android Version: Nougat 7.1.2

Additional context
This is currently fixed via manually editing the xml files to match the UI fields requested in the Java files above.

@minbi minbi self-assigned this Nov 21, 2018
@minbi minbi added bug Something isn't working cognito Issues with the AWS Android SDK for Cognito labels Nov 21, 2018
@mutablealligator mutablealligator added the mobile client Issues with AWS Mobile's client-side Cognito wrapper label Nov 26, 2018
@minbi minbi added pending-release Code has been merged but pending release and removed mobile client Issues with AWS Mobile's client-side Cognito wrapper labels Dec 5, 2018
@minbi
Copy link
Contributor

minbi commented Dec 5, 2018

Hi @dennisunderwood ,

Thanks for pointing out the issue. We are looking at it and aiming to resolve this in a future release.

@minbi
Copy link
Contributor

minbi commented Dec 6, 2018

Hi @dennisunderwood ,

We have released a fix in version 2.8.5 of the SDK. Please see if this fixes the issue for you.

@minbi minbi added pending-community-response Issue is pending response from the issue requestor and removed pending-release Code has been merged but pending release labels Dec 6, 2018
@minbi
Copy link
Contributor

minbi commented Dec 11, 2018

Hi,

We are closing this issue because there has been no activity. Please feel free to open a new issue if the problem persists. We ask this because closed issues are not actively monitored.

Thanks

@minbi minbi closed this as completed Dec 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cognito Issues with the AWS Android SDK for Cognito pending-community-response Issue is pending response from the issue requestor
Projects
None yet
Development

No branches or pull requests

3 participants