-
Notifications
You must be signed in to change notification settings - Fork 532
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
Implement Adobe IO provider #573
Conversation
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.
Please update the README to add the new provider to the table. There is a comment under the table containing a template for adding a new entry.
src/AspNet.Security.OAuth.AdobeIO/AdobeIOAuthenticationDefaults.cs
Outdated
Show resolved
Hide resolved
src/AspNet.Security.OAuth.AdobeIO/AdobeIOAuthenticationDefaults.cs
Outdated
Show resolved
Hide resolved
src/AspNet.Security.OAuth.AdobeIO/AdobeIOAuthenticationOptions.cs
Outdated
Show resolved
Hide resolved
src/AspNet.Security.OAuth.AdobeIO/AdobeIOAuthenticationOptions.cs
Outdated
Show resolved
Hide resolved
src/AspNet.Security.OAuth.AdobeIO/AspNet.Security.OAuth.AdobeIO.csproj
Outdated
Show resolved
Hide resolved
test/AspNet.Security.OAuth.Providers.Tests/AdobeIO/AdobeIOTests.cs
Outdated
Show resolved
Hide resolved
All done :) |
src/AspNet.Security.OAuth.AdobeIO/AdobeIOAuthenticationOptions.cs
Outdated
Show resolved
Hide resolved
src/AspNet.Security.OAuth.AdobeIO/AspNet.Security.OAuth.AdobeIO.csproj
Outdated
Show resolved
Hide resolved
…O.csproj Co-authored-by: Martin Costello <martin@martincostello.com>
Co-authored-by: Martin Costello <martin@martincostello.com>
Thanks for your contribution @biltongza 🎁 - could you verify it works in your application and with the real AdobeIO service using the If all working as expected, we can make a 5.0.8 release and push the provider to NuGet.org. |
I did a test of a local build before creating the PR, did another one now with the myget build and it's working fine. Do you need any evidence? |
Great thanks - no, I'll take your word for it 😃 It's just we've had a few new providers recently where they weren't tested with the real service by the contributors and we had to do patch releases the day after they first shipped. |
Implement Adobe IO authentication provider.
https://www.adobe.io/authentication/auth-methods.html#!AdobeDocs/adobeio-auth/master/AuthenticationOverview/AuthenticationGuide.md