-
Notifications
You must be signed in to change notification settings - Fork 154
Add support for Env and File OIDC. #977
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
tests/test_oidc.py
Outdated
assert source.id_token() == test_case.want | ||
|
||
|
||
# class MockIdTokenSource(oidc.IdTokenSource): |
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.
Is this left here on purpose?
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.
Nop, this should be removed, thanks!
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.
Approving with one comment
Signed-off-by: Renaud Hartert <renaud.hartert@databricks.com>
If integration tests don't run automatically, an authorized user can run them manually by following the instructions below: Trigger: Inputs:
Checks will be approved automatically on success. |
## Release v0.55.0 ### New Features and Improvements - Add support for OIDC ID token authentication from an environment variable ([PR #977](#977)). - Add support for OIDC ID token authentication from a file ([PR #977](#977)). ### Bug Fixes - Fix a reported highlighting problem with the way API clients are imported in WorkspaceClient/AccountClient ([#979](#979)).
What changes are proposed in this pull request?
This PR adds two new OIDC auth types,
env-oidc
andfile-oidc
, respectively extracting ID token from an environment variable or a file. This PR also opportunistically improves formatting and some imports.How is this tested?
Unit tests.