-
Notifications
You must be signed in to change notification settings - Fork 200
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
Ideally, Amplify/Cognito should not use a web view for Sign in with Apple #1121
Comments
I came across this Auth0 documentation on how they implement Sign in with Apple without a browser-based flow. It shows the sequence diagram that could work. Obviously this would be a lot more effort on your part. |
It would be great if they support this.
this behaviour is really annoying 👍 |
Thanks for the feedback. We'll take this on as a feature request and discuss with the Cognito team. As you identify, it would be a fairly different type of flow from the standard OAuth flow. We'll update this issue if and when we have any information to share. |
+1 to this. Somewhat related.. currently we have a need to clear all authentications on a fresh install and if a user signs in with apple with the web ui with |
Any update on this issue? This is very important and i've seen a lot of other Github posts about this specific issue, even with some users having their app rejected by Apple for using a web view. I've been looking for a solution everywhere, and all i've seen is the Amplify team saying they will consider this feature (however i've seen posts dating back to two years ago stating this.) |
We don't have any updates to report at this time.
I'm aware of one reported app rejection because of a blank page, but not specifically because of using a web view. Can you add some links to the GitHub posts you're referring to? |
This issue is stale because it has been open for 14 days with no activity. Please, provide an update or it will be automatically closed in 7 days. |
+1 Getting the same issue. Its been a long time, Any update on this? |
any updates? |
Any updates? |
2 similar comments
Any updates? |
Any updates? |
We don't have any updates to report on this issue at the moment. We are discussing the feature request with Cognito team and will post an update on this issue. |
Any updates on this issue? |
Hello @luiabrah - we are still discussing this with the Cognito team, we will provide updates when we are confident of our next steps. |
Any updates on this? I've read about the escape hatch solution, but not sure if after using the escape hatch, Amplify would have a session? Thanks! |
Amplify would have a valid session object containing Identity Id and Temporary AWS Credentials which is federated into the Identity Pools. Amplify would not have any details about the User Pool Tokens because the API federates into Identity Pools and NOT User Pools. |
Any updates on this? This issue is 3 years old now. This is something that should have been implemented a long time ago, no reason why sign-in with Apple has to be a terrible looking web ui for user pools. |
The inclusion of this feature in |
Any update? Looking forward this being implemented, especially on Mac where it does not ask for your fingerprint. |
I have impletemented Apple SignIn in my app, then I called "plugin.federateToIdentityPool()" and it is successful and I am able to fetch the credentials as well. BUT, when I called Amplify.Storage.uploadData(); I got error ""AuthError: Users Federated to Identity Pool do not have User Pool access.\nRecovery suggestion: To access User Pool data, you must use a Sign In method". Could you please help? I only wanted to have Apple SignIn in my app and do not want to create a custom SignIn flow or use User Pool. |
I believe this is a cognito feature request, more than it is an amplify request. I don't see a way to authenticate through direct cognito calls at all using apple tokens, which is what amplify would use. what amplify COULD do however, is to create a custom flow that accepts apple tokens. and create this feature for customers, before they all leave cognito/amplify for something that actually can pass IOS app store requirements (no web ui sign in is allowed) Where is the customer obsession and insists on highest standards? this is clearly not meeting the bar. |
Update: We have not yet received an update from the Cognito service team regarding the prioritization of the feature request. Please be assured that the Amplify team is in contact with them and is actively advocating for the prioritization of this request. We are committed to keeping you informed and will post any new information as soon as it becomes available. |
This comment from @palpatim mentions
Is this something that might work? or that would not be successful and that's why there is no follow-up on it? |
@CyprienRicque would you be able to share what you are trying to do? If you can share the use case, may be I can try providing an updated recommendation of what are your possible options. |
Thank you for your reply! Currently I am trying with custom lambda triggers. Also I wonder, even if it ends up working, would a user then be able to login to the same account with the hosted ui apple login and my custom setup? If you have some demo code for this method I am interested! |
I think this error is returned, may be the Lambda has not been setup correctly.
The solution looks good, although I would make sure all the edge cases have been accounted for. Using Custom Auth is the only way to achieve this solution without direct support from Cognito. |
Describe the bug
Amplify/Cognito use a Web View when displaying the Sign in with Apple native action sheet. Ideally, it would not use this at all.
To Reproduce
Steps to reproduce the behavior:
A web view is shown that is loading appleid.apple.com, and then the native "Sign in with Apple" action sheet overlay is shown.
Expected behavior
The web view would not be shown at all, it would directly go to the native sheet.
Environment(please complete the following information):
~/.amplify/logs/amplify-cli-<issue-date>.log
Device Information (please complete the following information):
Additional context
The mechanism for this is described in this comment by @palpatim. This could arguably be considered the same bug.
I'm curious if there is a way to use Sign in with Apple without needing to use
ASWebAuthenticationSession
at all. It leads to a sub-par user experience, and other problems like the blank web view when you cancel the Sign in with Apple process as described in #1027. This would of course necessitate a different API thansignInWithWebUI
.When I use a native SwiftUI
SignInWithAppleButton
the web view isn't shown at all. I'm not sure if there's a way for Cognito to "intercept" this though.The text was updated successfully, but these errors were encountered: