-
-
Notifications
You must be signed in to change notification settings - Fork 184
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
Native Sign In #5
Comments
+1, I have been waiting for this implementation. |
There needs to be some work done on the server side, but once that is done, we should be able to add this feature! |
Depend on supabase/auth-js#169 |
Made a PR to bring this to Dart |
Hello, I've implemented Sign In with Apple in my App, but on iOS instead of opening the sign in sheet in the app, it opens safari with Sign In with Apple the then the user is redirect in the app, I'm calling the function like this: supabase.auth.signInWithProvider( Provider.apple, options: AuthOptions( redirectTo: 'io.supabase.flutterquickstart://login-callback/'), ); Is it possible to open the sheet directly in the app and ket the user sign in, I've tried getting the credential and then passing to su-abase, but I always got an error of missing 'expires_in' parameters. I'm using su-abase 0.3.6 version on flutter |
When will it be implemented |
Hello, if you have any solutions for this i take it :) |
@dshukertjr hey :) Do you have even a little progress? We all waiting for it ) |
Thanks everyone for waiting patiently for this feature. Security comes first at Supabase, and we are making sure this feature lands securely when it does. Because of that, it is taking time. Again, thank you all so much for the patients. |
Hey @dshukertjr I have too ask you again about approximate TIME of this feature, because we should decide start it with firebase or wait for supabase. Thank you! |
As far as I can tell (and by the context of this issue and the library itself), native sign in has already been achieved with I also recall some issues with it on iOS. Currently, the bug-fixes are a priority, but we're looking forward to implement this feature for the stable 1.0. |
@dshukertjr I don't see it in supabase_flutter :( |
@dshukertjr Oh, that's very good!
|
I'm doing in that way but always get null 🤔 |
|
@dshukertjr do you have even rest api for creating user? So we can write some cloud function or do it in my backend? |
@bdlukaa maybe you can help me with openID api? Whatever I do, it returns null and does not create new users! The project's progress is stopped because of auth( |
I'm confused a little. We can do the auth today but it is via web, but the way to do it natively exist but isn't being documented & endorsed ? |
@ZetiMente The current implementation is half baked, and there needs to be some additional work to be done on the backend. |
Hey @dshukertjr, thanks for the video and article! Do you plan to support google_sign_in library in the future? It would save a lot of boilerplate. |
@ozasadnyy Hopefully yes, but the iOS version of Google sign in library has an issue where it does not allow the user to set their own nonce, which creates an security issue. Once this is fixed, developers can use the google sign in library with Supabase. |
@dshukertjr After adding sign in with google to my Flutter app (iOS) and signing in with the same account I use for sign in with apple I started getting this error on Sign in with Apple (iOS)
|
@mohsin2596 |
@dshukertjr I can confirm that auth providers on dashboard are setup correctly. This error only popped up for me when I updated Supabase to the latest package and allowed users to use Sign in with Google along with Sign in with Apple |
@mohsin2596 From your Supabase dashboard, could you take a look at the auth logs under |
|
Has anybody come across this issue? My app has been rejected by app review and I cant move forward without having it working Edit: |
@mohsin2596 |
@mohsin2596 |
@dshukertjr done. Will update here if it was instance related |
@dshukertjr After disabling and enabling the provider again it started working |
So, on Friday (apx 8/25) my Supabase instance started rejecting Sign In With Apple (native dialog). The relevant section in my logs was this one (redacted IP address):
At first I thought this was an issue with Apple but it turned out to be Supabase. After a weekend of not having Sign in with Apple working I found this thread and flipping it off and back on again everything started working again. Not sure what happened on the server side but ugh. :/ |
@mohsin2596 @wiverson Thanks for reporting. We are looking into it. |
Any update on this? Same error is happening when attempting to perform Google sign-in in Swift. |
@bitnom Make sure you have comma separated values under Authorized Client IDs in Google Provider for both iOS and Android. |
FWIW no update on my end other than after flipping off & on again things still seem to be working. |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This worked for me 🤷🏻♂️ |
I'm facing this issue and tried several approaches unsuccessfully. Google provider is enabled in dashboard and I copied over my Android client id in the
I feel like I'm running out of ideas as to what to do. I tried disabling/enabling provider to no success. Dashboard logs show this error.
|
Confusingly, you need to copy over the web client ID into your Supabase dashboard, and not the Android client ID. We can probably update the readme to emphasize more on this. |
@dshukertjr Thanks, can confirm this fixed my issue! |
@dshukertjr same....here |
Feature request
Is your feature request related to a problem? Please describe.
I want to sign in using a sign in provider, but the current implementation is not good. Launching the browser is not a good user experience.
Describe the solution you'd like
Use the
google_sign_in
package to sign the user in using Google andsign_in_with_apple
to sign the user in using Apple.Both of the packages return the user credentials, so, if I could sign in using the user credentials (just like Firebase does), it'd be great!
Aditional context
If launching the brower will still be the current solution, we could have in-app-handling.
By default, Android opens up a browser when handling URLs. You can pass
forceWebView: true
parameter to tell the plugin to open a WebView instead.The text was updated successfully, but these errors were encountered: