-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Proposal: Use Broadcast Channel for Apple Authentication in Android 🍎 #70
Comments
Hello, although up until today I have never thought about Firstly, Apple login on Android will always require a backend server. Period. This is a STRICT requirement from Apple. I understand that it may be inconvenient, however it often simplifies the login for the end user. Before intents, this plugin was using a webview. Using a webview was less secure; however, it was easier to implement in any app as it did not require custom intents. However, using a webview did not allow the usage of logging in via a device (instead of a password) or saving the password. Currently, the plugin opens the login page in Chrome. This means that we have no control over the behavior of the browser. Please let me know if a broadcast will redirect the user back into our app from Chrome. If it does, I would love to explore this further. However, if a broadcast is not possible to be send from Chrome or if it doesn't redirect the user back to the app, then I do not see a place for it in the codebase. |
From reading: And we can use postMessage there: |
This looks promising, I will admit that I have never heard of trusted web activity |
I'd like to suggest using Broadcast Channel for handling the client-server communication during Apple Sign-In authentication, rather than the current approach which requires setting up intents and server-side URL binding.
Current Implementation:
The existing solution requires:
Proposed Change:
Using Broadcast Channel could offer several benefits:
Implementation notes:
Happy to hear your thoughts on this approach! 💭
The text was updated successfully, but these errors were encountered: