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

Identity Pool is added when adding redirect URIs #4657

Closed
andrewbtp opened this issue Jun 24, 2020 · 3 comments · Fixed by #7255
Closed

Identity Pool is added when adding redirect URIs #4657

andrewbtp opened this issue Jun 24, 2020 · 3 comments · Fixed by #7255
Assignees
Labels
auth Issues tied to the auth category of the CLI bug Something isn't working referenced-in-release Issues referenced in a published release changelog

Comments

@andrewbtp
Copy link

andrewbtp commented Jun 24, 2020

Describe the bug
I set up auth with Cloud API only, an Identity Pool was not created. I then added redirect URIs to OAuth and suddenly it deployed an Identity Pool.

Amplify CLI Version
4.21.3

To Reproduce

Do you want to use the default authentication and security configuration? Manual configuration
 Select the authentication/authorization services that you want to use: User Sign-Up & Sign-In only (Best used with a cloud API only)
 Please provide a friendly name for your resource that will be used to label this category in the project: btpauth
 Please provide a name for your user pool: btpauth_pool
 Warning: you will not be able to edit these selections.
 How do you want users to be able to sign in? I want to learn more.

Selecting 'Email' and/or 'Phone Number' will allow end users to sign-up using these values.  Selecting 'Username' will require a unique username for users.


 How do you want users to be able to sign in? Email or Phone Number
 Do you want to add User Pool Groups? No
 Do you want to add an admin queries API? No
 Multifactor authentication (MFA) user login options: OFF
 Email based user registration/forgot password: Enabled (Requires per-user email entry at registration)
 Please specify an email verification subject: Your verification code
 Please specify an email verification message: Your verification code is {####}
 Do you want to override the default password policy for this User Pool? No
 Warning: you will not be able to edit these selections.
 What attributes are required for signing up?
 Specify the app's refresh token expiration period (in days): 30
 Do you want to specify the user attributes this app can read and write? No
 Do you want to enable any of the following capabilities?
 Do you want to use an OAuth flow? I want to learn more.

When you create a user pool in Amazon Cognito and configure a domain for it, Amazon Cognito automatically provisions a hosted web UI to let you add sign-up and sign-in pages to your app.


 Do you want to use an OAuth flow? Yes
 What domain name prefix do you want to use? xx
 Enter your redirect signin URI: http://localhost:3000/
? Do you want to add another redirect signin URI No
 Enter your redirect signout URI: http://localhost:3000/
? Do you want to add another redirect signout URI No
 Select the OAuth flows enabled for this project. Authorization code grant
 Select the OAuth scopes enabled for this project. Phone, Email, OpenID, Profile, aws.cognito.signin.user.admin
 Select the social providers you want to configure for your user pool: Facebook, Google

 You've opted to allow users to authenticate via Facebook.  If you haven't already, you'll need to go to https://developers.facebook.com and create an App ID.

 Enter your Facebook App ID for your OAuth flow:  xx
 Enter your Facebook App Secret for your OAuth flow:  xx

 You've opted to allow users to authenticate via Google.  If you haven't already, you'll need to go to https://developers.google.com/identity and create an App ID.

 Enter your Google Web Client ID for your OAuth flow:  xx
 Enter your Google Web Client Secret for your OAuth flow:  xx
? Do you want to configure Lambda Triggers for Cognito? No

amplify push
and then

 What do you want to do? Add/Edit signin and signout redirect URIs
 Which redirect signin URIs do you want to edit?
 Do you want to add redirect signin URIs? Yes
 Enter your new redirect signin URI: https://dev.x.co/
? Do you want to add another redirect signin URI No
 Which redirect signout URIs do you want to edit?
 Do you want to add redirect signout URIs? Yes
 Enter your new redirect signout URI: https://dev.x.co/
? Do you want to add another redirect signout URI No

amplify push

Expected behavior
I expect no Identity Pool to be created.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: Mac
  • Node v10.15.1

Additional context
Add any other context about the problem here.

@akshbhu akshbhu added pending-triage Issue is pending triage auth Issues tied to the auth category of the CLI labels Jun 24, 2020
@akshbhu akshbhu self-assigned this Jun 29, 2020
@edwardfoyle edwardfoyle assigned edwardfoyle and unassigned akshbhu Aug 14, 2020
@edwardfoyle
Copy link
Contributor

Hi @andrewbtp I was able to repro this. Marking as a bug

@edwardfoyle edwardfoyle added bug Something isn't working and removed pending-triage Issue is pending triage labels Aug 21, 2020
@attilah attilah assigned attilah and unassigned edwardfoyle May 4, 2021
@github-actions github-actions bot added the referenced-in-release Issues referenced in a published release changelog label May 14, 2021
@github-actions
Copy link

👋 Hi, this issue was referenced in the v4.51.0 release!

Check out the release notes here https://github.com/aws-amplify/amplify-cli/releases/tag/v4.51.0.

@github-actions
Copy link

This issue has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs.

Looking for a help forum? We recommend joining the Amplify Community Discord server *-help channels for those types of questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auth Issues tied to the auth category of the CLI bug Something isn't working referenced-in-release Issues referenced in a published release changelog
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants