-
Notifications
You must be signed in to change notification settings - Fork 542
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
Update README.md to recommend using OpenIddict for new applications #838
Conversation
@@ -1,12 +1,42 @@ | |||
|
|||
# AspNet.Security.OAuth.Providers | |||
|
|||
**AspNet.Security.OAuth.Providers** is a **collection of security middleware** that you can use in your **ASP.NET Core** application to support social authentication providers like **[GitHub](https://github.com/)**, **[Foursquare](https://foursquare.com/)** or **[Dropbox](https://www.dropbox.com/)**. It is directly inspired by **[Jerrie Pelser](https://github.com/jerriep)**'s initiative, **[Owin.Security.Providers](https://github.com/RockstarLabs/OwinOAuthProviders)**. | |||
**AspNet.Security.OAuth.Providers** is a **collection of security middleware** that you can use in your **ASP.NET Core** application to support social authentication providers like **[GitHub](https://github.com/)**, **[Twitter/X](https://twitter.com/)** or **[Dropbox](https://www.dropbox.com/)**. It is directly inspired by **[Jerrie Pelser](https://github.com/jerriep)**'s initiative, **[Owin.Security.Providers](https://github.com/RockstarLabs/OwinOAuthProviders)**. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: I replaced Foursquare by Twitter because their new API no longer supports OAuth 2.0 (they now use API keys).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My only comment is that it's quite big for a "tip" - on my screen setup at least, it's the almost the entire height of the screen.
Maybe just have a smaller tip at the top, that then links to a section further down with the additional details?
29eb324
to
dd9fb84
Compare
Good idea! 👍🏻 |
Looks good 👍 |
Thanks! |
With most of the pain points addressed in previous OpenIddict releases (e.g WS-Federation claims or authentication schemes mapping) and a lot of services now being supported (78 at the time of writing), the OpenIddict client and its web providers are a good alternative to the packages we maintain here.
As such, I'd like to update the README to encourage users to adopt them for new applications (while still making clear the aspnet-contrib providers are still supported, of course).