Skip to content

feat: Python auth local server - Add new GOOGLE_MANAGED_KAFKA_AUTH_PRINCIPAL environment variable that allows users to provide the auth principal for credential types where it cannot be programmatically fetched#49

Merged
davidtorres merged 5 commits into
googleapis:mainfrom
benjamin-maynard:python-support-principal-env
May 20, 2026

Conversation

@benjamin-maynard
Copy link
Copy Markdown
Contributor

@benjamin-maynard benjamin-maynard commented Oct 11, 2025

Add new GOOGLE_MANAGED_KAFKA_AUTH_PRINCIPAL environment variable for principal override. This allows users to provide the auth principal for credential types where it cannot be programmatically fetched.

This allows the auth server to be used with BYOID credential types such as Workforce Identity Federation.

Also dropped 2 imports that are not used and added requests which the auth library needs for fetching some BYOID tokens.

Googlers see b/449376528#comment13 for more details.

@benjamin-maynard benjamin-maynard requested a review from a team October 11, 2025 13:47
@benjamin-maynard benjamin-maynard force-pushed the python-support-principal-env branch from 41a37a8 to 105228f Compare October 11, 2025 15:07
…RINCIPAL` environment variable that allows users to provide the auth principal for credential types where it cannot be programmatically fetched
@benjamin-maynard benjamin-maynard force-pushed the python-support-principal-env branch from 105228f to 569c4cc Compare October 11, 2025 15:08
Comment thread kafka-auth-local-server/kafka_gcp_credentials_server.py Outdated
Comment thread kafka-auth-local-server/kafka_gcp_credentials_server.py Outdated
@benjamin-maynard benjamin-maynard requested a review from a team as a code owner May 16, 2026 17:20
Comment thread kafka-auth-local-server/kafka_gcp_credentials_server.py Outdated
@davidtorres davidtorres merged commit acf5403 into googleapis:main May 20, 2026
4 checks passed
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