-
Notifications
You must be signed in to change notification settings - Fork 18
feat(icp4d): refactor iam token manager and add icp4d support #14
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
Conversation
1ad372a
to
91e6e9a
Compare
91e6e9a
to
c536b63
Compare
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.
I think this looks good overall, just a couple of changes and quetions
9a0eb6d
to
66da17e
Compare
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.
👍 I think this looks good. A few comments but nothing major!
501a2ad
to
f68f00c
Compare
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.
👍 looks good!
This PR refactor how we handle IAM tokens and adds separate token managers to handle token operations.