Add support for Okta without breaking backwards compatibility for OneLogin

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

Add support for Okta without breaking backwards compatibility for OneLogin #48

Closed punmechanic closed 3 years ago

punmechanic commented 3 years ago

adds Okta support
adds additional command line flags for improved developer and user experience such as --hostname, --provider and --role

this requires #47 to be merged before being merged

This is still WIP and, obviously, it's around +1500 without #47 so there are more changes than enumerated here. I'm hoping to keep the diff around +1500 if possible after all testing changes are removed and I'll exhaustively enumerate all changes in this PR

punmechanic commented 3 years ago

This is in a place now where it could be used in a beta testing group.

Okta is now supported alongside OneLogin and older versions of KeyConjurer will still work with the new API. It's ready for code review.

punmechanic commented 3 years ago

This will be superseded by a PR that is broken up a bit more to be less overwhelming.