-
Notifications
You must be signed in to change notification settings - Fork 16
Open
Description
Right now the auth plugin has lots of loginWith methods.
loginWithGoogle()
loginWithMeta()
loginWithApple()
loginWithTwitter()
loginWithGithub()
loginWithLinkedin()
loginWithMicrosoft()
Which might grow as soon as we want to add a new provider method. And this functionality is "closed"(in OCP terms), but not open for adding new providers.
A better way would be to open the plugin interface for adding a provider implementation, and accept a contract for login, logout etc.
This way the core plugin contract will not have to worry about adding new providers, as and when required, keeping the abstractions at the heart and extensibility at the consumer end.
Metadata
Metadata
Assignees
Labels
No labels