-
Notifications
You must be signed in to change notification settings - Fork 73
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
The IDP has to support additional infrastructure to support FedCM #441
Comments
One way forward would be to expand upon the IDP Sign in status API and have a top level IDP window get opened. I discussed something like this in fedidcg/proposals#3 but it could be adapted here. I'll add a comment with a general shape of the proposal. |
How this could work: We need to allow the FedCM n.c.get API to open a window. Let's add to the This new config would be defined as
If a IdentityProviderNavigationConfig is provided, then the browser should not try to access any IDP APIs. Instead, it should display the IDP using The IDP sign in status API is augmented to be the following:
This also requires a per-origin map for IDP pending login requests. It should be added to when the When If we decide that IdentityCredentials are storable, we should also extend I'm eliding a lot of detail here. And it will depend on the resolution of #436 and #438 and #439. However, I think this sketches how we can skirt around the IDP API requirement via an opener and using the IDP Sign in Status API as a communication channel. |
Specifically, the items in Identity Provider HTTP API. It would be nice to eliminate this requirement.
The text was updated successfully, but these errors were encountered: