Skip to content

Document all Google Cloud connection fields - #76160

Merged
shahar1 merged 1 commit into
apache:mainfrom
pavansai1997:docs-google-connection-fields
Oct 31, 2026
Merged

shahar1 merged 1 commit into
apache:mainfrom
pavansai1997:docs-google-connection-fields

Conversation

@pavansai1997

@pavansai1997 pavansai1997 commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor

The Google Cloud connection page did not describe five fields shown in the connection form
(anonymous credentials and the four fields of the external IdP Client Credentials Grant
flow), and the list of keys accepted when defining the connection as a URI or JSON stopped
at num_retries. Users configuring the connection through environment variables or a
secrets backend had to read GoogleBaseHook to discover credential_config_file,
impersonation_chain, quota_project_id, is_anonymous, idp_issuer_url, client_id,
client_secret and idp_extra_parameters.

The new descriptions follow GoogleBaseHook and _CredentialProvider, including which key
options are mutually exclusive with anonymous credentials.

The hook currently ignores idp_extra_parameters; the companion PR "Fix Google Cloud
connection IdP extra request parameters being ignored" fixes that, so it is best merged first.


Was generative AI tooling used to co-author this PR?
  • Yes — Code Puppy

Generated-by: Code Puppy following the guidelines

Five fields shown in the Google Cloud connection form, covering anonymous
credentials and the external IdP Client Credentials Grant flow, had no
description, and the list of keys accepted when defining the connection
as a URI or JSON stopped at num_retries. Users configuring the
connection from environment variables or secrets backends had to read
the hook source to discover those keys.
@pavansai1997
pavansai1997 requested a review from shahar1 as a code owner October 31, 2026 17:56
@boring-cyborg boring-cyborg Bot added area:providers kind:documentation provider:google Google (including GCP) related issues labels Oct 31, 2026
@shahar1
shahar1 merged commit 7b47022 into apache:main Oct 31, 2026
62 of 63 checks passed
@boring-cyborg

boring-cyborg Bot commented Oct 31, 2026

Copy link
Copy Markdown

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers kind:documentation provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants