Closed npm1 closed 11 months ago
Ping
lgtm
LGTM
@bvandersloot-mozilla WDYT?
I think I follow and LGTM.
I think I follow and LGTM.
Thanks @bvandersloot-mozilla ! @npm1 i'll wait for you to resolve the last remaining bits of feedback from @TallTed and then squash and merge.
Fixes https://github.com/fedidcg/FedCM/issues/496. Adds a
disconnect_endpoint
and adisconnect
method to allow the RP to request the IDP to disconnect a previously connected federated account. I opted fordisconnect_endpoint
instead ofdisconnection_endpoint
because it sounded a bit better to me, but open to change it. The credentialed request is sent with IDP cookies but using CORS as the request mode. If no account is found or there is some other error, the user agent removes all connections associated with the RP, IDP.Preview | Diff