-
Notifications
You must be signed in to change notification settings - Fork 20
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
WIP OAuth2 Support #165
WIP OAuth2 Support #165
Conversation
Wow, thank you @bstopp !!! @francoisledroff is out this week, but will review when he is back. |
Remove JUnit from modules without tests.
@francoisledroff - I don't know why the integration tests are failing. Its saying NPE, which indicates it can't find the fallback properties file. The actions should have all the necessary properties defined to successfully run these checks. I don't have the information necessary to run the IT tests offline. I'm not sure if this is something you can do, to check the updates and make sure it wasn't something i introduced? |
I went ahead and updated all the plugins and dependencies to latest releases. I'll be going back and adding more tests over the next couple of days while folks find time to review. |
@bstopp the IT tests are leveraging GH secrets only available on the main remote repo. |
Closing this in favor of #166 |
Description
Update API to support OAuth Server-to-Server client credentials approach to Authentication.
Related Issue
#155
Motivation and Context
Updates library to prepare for removal of deprecated JWT solution.
How Has This Been Tested?
Update existing unit tests, and added new ones.
Screenshots (if appropriate):
Types of changes
Checklist: