RiotGames / key-conjurer

Temporary Credential Service
https://technology.riotgames.com/news/key-conjurer-our-policy-least-privilege
Apache License 2.0
167 stars 33 forks source link

Okta #50

Closed punmechanic closed 3 years ago

punmechanic commented 3 years ago

This PR addresses all the changes required for supporting Okta within KeyConjurer. The README should be ignored as while it is partially up to date it hasn't been updated due to a focus on functionality. I've left comments inline of decisions made if they aren't obvious or seem strange.

Also please do not be intimidated by the file count. A large number of those files were me moving files from one folder to another. This was done in order to better share code between the client and the server or to reduce unnecessary nesting.

That said, this PR was always destined to be large, and was around +2500-2000 before any of the auto-formatting and colocation of code because the changes that needed to be made essentially touched every aspect of the application.