-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Ability to customize pop ups when calling signInWithRedirect on iOS #3410
Comments
@mkrn do you get the same alert when you logout/signout of your app? I get the same alert “... wants to use Amazoncognito to signin...” both when the user signs in and when the user signs out. Different problem, but just in case you have not tested the signout flow. |
@bonillam Yes, get the same alert on sign out: |
@mkrn I do not think the flow you are describing is possible. There's a statement buried in the docs indicating that RN apps must use the hosted Cognito UI (which you can hide), but the initial web browser redirect does go there before Facebook. The popup is an Expo thing indicating an oauth flow via a web redirect is about to happen. https://docs.expo.io/versions/v30.0.0/sdk/auth-session/ One small idea for an improvement would be to associate a custom domain with Cognito in the AWS Console, so at least the user associates the auth flow with an existing app experience. |
Thanks, @loganwedwards As for the second part of my issue, I think that even though Expo web browser shares cookies with system browser, not all users are logged in to Facebook in the browser, but most are in the Facebook app. I will explore integrating Amplify iOS SDK with FB SDK since I already have swift parts as well in the app. |
+1 , is there another url opener we can provide to the was config so we can customize the " "appName" Wants to Use "amazoncognito.com" to Sign In " message, which is shown for both sign in and sign out ? instead of using the default Expo Link opener provided here : https://aws-amplify.github.io/docs/js/authentication#a-note-for-expo-users |
Closing this issue as it seems it was solved. Please feel free to reopen in need be. |
The last comment is an open, unanswered question, and the issue is not in any way fixed, and no workaround or solution was offered, so why on earth would you say it is solved? I'm facing the same problem. When a user tries to sign out on iOS, they get a pop up saying my app wants to use my domain to sign in. A user should never have to give permission just to sign out. How to I circumvent this? |
Apologizes, I read it as the workaround was able to resolve this. Reopening issue |
@mkrn Does it happen if we config app scheme in production? |
I am currently seeing the same issue with logout displaying the "sign-in" system alert. Can we either cache that a user has already provided permission to visit this URL on sign-in (thus making the second alert on sign-out obsolete), change the contents of the message so it does not indicate sign-in, or remove the need for it altogether? |
Is there any update/workaround regarding this issue? |
Having the same problem even with a custom scheme. Compared with some existing user experiences of other apps using federated sign in (Quora for example) and none of them have this experience especially a text describing "'XYZ' wants to use 'ABC' to sign in" on logout. This is so confusing and misleading. Can we please have a workaround at least? |
Hello!! Has there been any progress here? Again, I would love for amplify to to say "sign out" in the alert when I call the sign out function. |
Just chiming in as well. Our app is also affected by this UX issue. Thanks! |
We are using hosted UI and native swift and this is a super annoying and confusing user experience. Is there a workaround to solve this issue? |
@ahmadfs We are investigating a custom react ui that acts as a login proxy. Instead of calling amplify from our mobile app, we will open our custom react hosted ui in an inapp browser and call the amplify login function from there and pass the jwt token back to the mobile app via deep linking. Then when we sign out we will delete the jwt from the application. |
This is a good enough workaround, but still not perfect |
It's been more than 3 years since this issue has been opened. It is clearly impacting a lot of amplify customers. Yet we have not heard anything from the amplify team on prioritizing / fixing this issue. Anyone who uses social sign on, one of the most basic app features is affected by this issue. We aren't asking for rocket science we just need to change the super confusing sign in and sign out dialog's. The Amplify team needs to pause whatever they are doing and focus on fixing the HUGE backlog of issues customers are facing. I don't have many regrets in life, but deciding to use amplify for my app is now one of them. |
If it's not too late for you, I'd say move on from Amplify and use React Native Firebase. It has plenty of wonky changes you may have to make to your Podfile, but I implemented oAuth in a night compared to two days with Amplify. Check out all of these comments of users totally frustrated with Amplify. I feel as though this should not be a product if they are going to undeserve customers in this manner. |
@sidhantchadda @AlexChaseJones is the consensus here that we want to suppress this message or change what it displays? |
Ideally suppress, if that is not possible, we can change what it displays, to include ex: '"appname" wants to use facebook to sign in'. |
thank you for the quick response, we are trying to prioritize this and wanted to make sure on the desired behavior before we address it |
Is there any update if this message can be suppressed? @tannerabread |
Hi @dgrosch02 - unfortunately, that is not currently possible. We are working with the Amazon Cognito team to allow you to pass an Authentication token to authorize access from social providers rather than using the hostedUI, which would provide you with much better control over your end user experience. We will provide updates here when we have an ETA! |
If anybody is using expo-web-browser to do the url redirects for the auth config, you can add additional options to stop the pop up. For me: |
Hi, any update about this issue ? |
With expo-web-browser I just did:
Because using |
Unfortunately |
I guess for now i'll just hack the signout so that it doesn't show that alert when calling signOut that says the app wants to redirect to website to sign in.
|
@abdallahshaban557 any update on the timeline? half a year seems already pretty long. It would be awesome to finally be able to use the same pattern as with federated sign in, when using cognito user pools. |
Explicitly calling
|
@banghia112 The code provided doesn't actually terminate the session with the OAuth provider, so when logging in again, the account selection screen might not appear as expected. This is because ending the session with the OAuth provider, thus prompting the account choice upon the next login, requires a logout from the OAuth provider's side. Have you found a way to end the session with the OAuth provider? |
any update? sept 2024... |
Waiting for updates Oct 2024 |
Describe the bug
Using withOAuth HOC and facebookSignIn flow in expo bare / react native shows alert that says "AppName Wants to Use "amazoncognito.com" to Sign In"
The users may not be familiar with amazoncognito.com and may be confused since they chose to sign in with Facebook, not Amazon.
New users need to sign in to facebook even if they're signed in to Facebook App on their iPhone. This lowers conversions since users have to enter email/password that they may not remember.
Is there any way to have permission say "use 'facebook.com'" to sign in?
Is there any way to have new users signed in if they've signed in to the Facebook app on their phone?
I'm using expo bare and already have Facebook SDK Pod, so if there's a native way to achieve Cognito+Facebook sign in please share.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):
Additional context
Add any other context about the problem here.
Sample code
You can turn on the debug mode to provide more info for us by setting window.LOG_LEVEL = 'DEBUG'; in your app.
The text was updated successfully, but these errors were encountered: