Implement support for MFA (SMS or email) when logging in. Also update dependencies (ng 5.1.0, etc).

[amirtoole]

This resolves #121

[joe455]

@hellofornow I followed the same steps but getting error.

callback.mfaSetUp is not a function.

Any Idea?

[amirtoole]

Did you run npm install before trying this PR? I did update the amazon-cognito-identity-js dependency (taking a guess that's the problem).

[joe455]

@hellofornow I found the route cause finally.

• I am not saving and confirming phone number.
• Which is the mandatory process to use MFA functionality. ( Multi-Factor Authentication (MFA) increases security for your end users. If you choose 'optional', individual users can have MFA enabled. You can only choose 'required' when initially creating a user pool, and if you do, all users must use MFA. Phone numbers must be verified if MFA is enabled. You can configure adaptive authentication on the Advanced security tab to require MFA based on risk scoring of user sign in attempts.)

Also, I followed docs here and customised my code to verify both email and phone.

Thanks for the immediate response and your PR is working just fine with the latest version of cognito JS SDK.

Kudos @hellofornow for the PR

Let me know if you found any solution to add facebook user to user pool :p Happy weekend!!!

[joe455]

@hellofornow OnSubmit() is not getting called.

Use Submit instead of OnSubmit. 👍