-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat: bedrock token provider #3161
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
df114c1
to
1c651d3
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. Just some minor changes and some questions.
Thanks!
1c651d3
to
5dd1dbb
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.
Clean implementation. Just a question and two minor asks.
Thanks!
5dd1dbb
to
6053a51
Compare
Smoke test run:
Integ test run:
|
Description of changes:
Adds a provider to fetch token values from environment. If found, sets auth scheme preference to prefer
bearer
. Additional tests will be added once #3156 is approved and merged.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.