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

totpIssuer defaults to AWSCognito #6247

Open
4 tasks done
JamesToBrightest opened this issue Dec 12, 2024 · 4 comments
Open
4 tasks done

totpIssuer defaults to AWSCognito #6247

JamesToBrightest opened this issue Dec 12, 2024 · 4 comments
Assignees
Labels
Authenticator An issue or a feature-request for an Authenticator UI Component question General question

Comments

@JamesToBrightest
Copy link

JamesToBrightest commented Dec 12, 2024

Before creating a new issue, please confirm:

On which framework/platform are you having an issue?

React

Which UI component?

Authenticator

How is your app built?

Next.js

What browsers are you seeing the problem on?

Chrome, Firefox

Which region are you seeing the problem in?

eu-west-1

Please describe your bug.

totpIssuer in formFields not used and defaults to AWSCognito.

What's the expected behaviour?

Uses the totpIssuer field.

Help us reproduce the bug!

See code snippet.

Code Snippet

Authenticator:

            <Authenticator components={components} formFields={formFields} hideSignUp>
                {({ signOut, user }) => (
                    <GlobalContextProvider>
                        <React.Fragment>{children}</React.Fragment>
                    </GlobalContextProvider>
                )}
            </Authenticator>

FormFields:

const formFields = {
    confirmSignIn: {
        confirmation_code: {
            label: 'Code',
            placeholder: 'Code',
            isRequired: true,
            labelHidden: true,
        },
    },
    setUpTOTP: {
        QR: {
            totpIssuer: 'Test Issuer',
        },
    },
};

Console log output

No response

Additional information and screenshots

No response

@github-actions github-actions bot added pending-triage Issue is pending triage pending-maintainer-response Issue is pending response from an Amplify UI maintainer labels Dec 12, 2024
@cwomack cwomack added the Authenticator An issue or a feature-request for an Authenticator UI Component label Dec 12, 2024
@cwomack cwomack self-assigned this Dec 12, 2024
@cwomack
Copy link
Member

cwomack commented Dec 12, 2024

Hello, @JamesToBrightest and thanks for opening this issue. Cognito will be the default issuer and Authentication provider for all things related to Amplify, but are you trying to override it to be another issue? This may also be giving you an issue due to setupTotp needing to have those last 3 letters be lowercase (whereas in your example it looks like you have it as setUpTOTP). Can you see if updating this changes the behavior for you?

@github-actions github-actions bot removed the pending-maintainer-response Issue is pending response from an Amplify UI maintainer label Dec 12, 2024
@cwomack cwomack added question General question pending-community-response Issue is pending response from the issue requestor or other community members and removed pending-triage Issue is pending triage labels Dec 12, 2024
@cwomack
Copy link
Member

cwomack commented Dec 12, 2024

@JamesToBrightest
Copy link
Author

Hello, @JamesToBrightest and thanks for opening this issue. Cognito will be the default issuer and Authentication provider for all things related to Amplify, but are you trying to override it to be another issue? This may also be giving you an issue due to setupTotp needing to have those last 3 letters be lowercase (whereas in your example it looks like you have it as setUpTOTP). Can you see if updating this changes the behavior for you?

Hi, thanks for your response. Changing it to setupTotp did not resolve the issue as the issuer remains AWSCognito.

@github-actions github-actions bot added pending-maintainer-response Issue is pending response from an Amplify UI maintainer and removed pending-community-response Issue is pending response from the issue requestor or other community members labels Dec 13, 2024
@cwomack
Copy link
Member

cwomack commented Dec 19, 2024

@JamesToBrightest, I've been trying to reproduce this on my side with no luck. Do you have a minimal sample app or repo that you can share which reliably reproduces this? Thanks.

@github-actions github-actions bot removed the pending-maintainer-response Issue is pending response from an Amplify UI maintainer label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Authenticator An issue or a feature-request for an Authenticator UI Component question General question
Projects
None yet
Development

No branches or pull requests

2 participants