Repository navigation
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
…ovider-management-sso_admin
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
…ovider-management-sso_admin
Summary
Adds SAML single sign-on identity provider management to the admin client as
client.auth.admin.sso, covering theauth.sso_admincapability group.AuthAdminSSOApiwithlistProviders,createProvider,getProvider,updateProvideranddeleteProvider, against/admin/sso/providers.SAMLMetadatais a sealed class with.urland.xmlconstructors, so a request carries exactly one metadata source.SAMLNameIdFormatis an enum of the four formats the server accepts, encoded as their URNs.SSOProvider,SAMLProvider,SAMLAttributeMappingandSAMLAttributemodel the response.SAMLProvider.metadataXmlis nullable becauselistProvidersleaves it out.resource_<resourceId>addressing works, and they are URL encoded.Update semantics
The server applies a partial update, and
UpdateSSOProviderOptionsfollows its rules:domainsorattributeMappingkeeps the stored value. An empty list removes every domain and a mapping with no keys clears the mapping.nameIdFormatis the exception: the server clears it when it is not sent. The field is therefore required (nullable), so a caller passes the current format to keep it rather than wiping it by accident.Compliance matrix
auth.sso_admin.create_provider,get_provider,list_providers,update_provideranddelete_providerare registered asimplemented. The IDs were added incapability-matrix/v1.15.0, so the validation workflow pin moves from v1.14.0 to v1.15.0. That release also definesauth.admin.update_mfa_factor, which stays undeclared here and is reported as not implemented.Tests
test/admin_sso_test.dart: request and response serialization, and the requests each method sends, withMockSupabaseHttpClient.test/admin_test.dart: create, get (by resource id), list, update and delete against the local stack, including the keep and clear behavior of partial updates. SAML sign-in does not need to be enabled for the admin endpoints.Resolves SDK-2257
Summary by CodeRabbit