Closed bstopp closed 1 year ago
Wow, thank you @bstopp !!! @francoisledroff is out this week, but will review when he is back.
@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. If you push your change against a remote branch and update your PR accordingly, the IT test should run (using the deprecated JWT authentication)
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: