Skip to content

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

Merged
merged 12 commits into from
May 26, 2025
Merged

Conversation

renaudhartert-db
Copy link
Contributor

@renaudhartert-db renaudhartert-db commented May 23, 2025

What changes are proposed in this pull request?

This PR adds two new OIDC auth types, env-oidc and file-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.

@renaudhartert-db renaudhartert-db marked this pull request as ready for review May 23, 2025 18:27
assert source.id_token() == test_case.want


# class MockIdTokenSource(oidc.IdTokenSource):
Copy link
Contributor

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?

Copy link
Contributor Author

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!

Copy link
Contributor

@hectorcast-db hectorcast-db left a 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>
Copy link

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-py

Inputs:

  • PR number: 977
  • Commit SHA: 9c5090e5b1913d86c7836d447d1c073d9e8bfe8e

Checks will be approved automatically on success.

@renaudhartert-db renaudhartert-db added this pull request to the merge queue May 26, 2025
Merged via the queue into main with commit 9e96bac May 26, 2025
17 checks passed
@renaudhartert-db renaudhartert-db deleted the renaud-hartert_data/oidc branch May 26, 2025 10:19
deco-sdk-tagging bot added a commit that referenced this pull request May 27, 2025
## 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)).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants