Add appwrite OAuth2 provider to migration allow-list#207
Conversation
Appwrite server is adding a built-in 'appwrite' OAuth2 provider (sign in with Appwrite Cloud). Without an entry in the PROVIDERS allow-list, migrations report "No migration resource for OAuth2 provider 'appwrite'; skipped." Only clientId migrates (to the appwriteAppid attribute); clientSecret is write-only and stays redacted per the existing policy. The OAuth endpoint is hardcoded server-side, so there is no endpoint setting. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Greptile SummaryThis PR adds
Confidence Score: 5/5The change is a single allow-list entry and a new test file; no existing logic is modified. The addition follows the established pattern exactly, is placed correctly in alphabetical order, deliberately omits clientSecret (consistent with the class's redaction policy), and is covered by a well-structured test that also exercises secret-redaction across every provider in the list. No files require special attention. Important Files Changed
Reviews (1): Last reviewed commit: "Add appwrite OAuth2 provider to migratio..." | Re-trigger Greptile |
What
Adds the
appwriteprovider key (Appwrite-as-an-identity-provider, e.g. sign in with Appwrite Cloud) to theOAuth2Provider::PROVIDERSallow-list, inserted alphabetically betweenappleandauth0.Why
Appwrite server is adding a built-in
appwriteOAuth2 provider. Migrations that include theoauth2-providerresource currently fail with:Details
clientIdmigrates, targeting the destination'sappwriteAppidproject attribute — same shape asgithub/discord.clientSecretis write-only on the server and is never copied, consistent with the class's secret-redaction policy.https://cloud.appwrite.io/v1/oauth2/console), so there is no endpoint/domain setting to carry over.Sources/Appwrite.phporDestinations/Appwrite.php— both are driven by thePROVIDERSdescriptor.Tests
New
OAuth2ProviderTestunit test coveringfromArray()for theappwritekey, secret redaction across every provider in the allow-list, and unknown-provider handling.A release should follow this merge so appwrite/appwrite can bump
utopia-php/migrationpast 1.16.0.🤖 Generated with Claude Code