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

Not able to change password #8

Open
shantanuo opened this issue Mar 4, 2017 · 5 comments
Open

Not able to change password #8

shantanuo opened this issue Mar 4, 2017 · 5 comments

Comments

@shantanuo
Copy link

I tried Chapter 09 code and everything worked as expected. But I could not change password.
On the changePassword.html page, I get a message that user has logged in and then "Password not changed" message - instead of "Password changed".

@shantanuo
Copy link
Author

for the line 69, I get a tip in Amazon console that says "use === to compare with null"
if (correctHash == null) {

And I also get warning "Missing semicolon" on the last line of the code "sampleAuthChangePassword"

@shantanuo
Copy link
Author

The code for "Change Password" function is different than the one available in the zip file.
sampleAuthChangePassword

When I uploaded the zip file to lambda function, change password page worked as expected. I have created the pull request.

@shantanuo
Copy link
Author

There is 1 line missing in the file "Policy_Cognito_sampleAuth_Unauth_Role.json"

"arn:aws:lambda::<AWS_ACCOUNT_ID>:function:sampleAuthChangePassword"

Change password function needs access just like all other functions.

@shantanuo
Copy link
Author

Another problem with "Change password" link is that if you get redirected to S3 page that looks something like this...
https://testlogin721.s3.amazonaws.com/changePassword.html

You can not change password. You need to visit the ec2 instance page that looks like this...

http://ec2-34-200-222-179.compute-1.amazonaws.com/changePassword.html

And now you are able to change your password. Weird.

@danilop
Copy link
Owner

danilop commented Apr 19, 2018

There was an issue in the code, sorry for that. It should work now.

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

No branches or pull requests

2 participants