Skip to content

Document that Dag bundle kwargs should reference a Connection, not inline credentials - #68373

Merged
vatsrahul1001 merged 2 commits into
apache:mainfrom
potiuk:docs-dag-bundle-kwargs-reference-connections
Jun 28, 2026
Merged

Document that Dag bundle kwargs should reference a Connection, not inline credentials#68373
vatsrahul1001 merged 2 commits into
apache:mainfrom
potiuk:docs-dag-bundle-kwargs-reference-connections

Conversation

@potiuk

Copy link
Copy Markdown
Member

Adds guidance to the Dag bundles documentation that credentials for a Dag bundle should be supplied through an Airflow Connection (git_conn_id / aws_conn_id / gcp_conn_id) rather than inlined into the bundle kwargs.

dag_bundle_config_list is surfaced by the Config API when [api] expose_config is enabled, so a credential placed directly in a bundle's kwargs (for example a token embedded in repo_url) is readable by anyone authorized to read the configuration. Referencing a Connection keeps the credential in the secrets backend and out of the exposed configuration.

Docs-only change.

Was generative AI tooling used to co-author this PR?
  • Yes — Claude Opus 4.8 (1M context)

Generated-by: Claude Opus 4.8 (1M context) following the guidelines at
https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions

@potiuk
potiukforce-pushed the docs-dag-bundle-kwargs-reference-connections branch from fa4a96f to cb24ac7CompareJune 14, 2026 01:51
Generated-by: Claude Opus 4.8 (1M context)
@potiukpotiuk added this to the Airflow 3.3.0 milestone Jun 17, 2026
@potiuk

Copy link
Copy Markdown
MemberAuthor

I gues we can still merge this one before beta @vatsrahul1001 ?

@potiuk
potiuk requested a review from jschefflJune 17, 2026 02:51
@potiuk

Copy link
Copy Markdown
MemberAuthor

Small doc update @vatsrahul1001 -> just enough to clarify potential user misconfiguration of kwargs for passing credentials.

@vatsrahul1001vatsrahul1001 added the backport-to-v3-3-test Backport to v3-3-test label Jun 28, 2026
@vatsrahul1001
vatsrahul1001 merged commit 8c74f8e into apache:mainJun 28, 2026
69 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-3-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

StatusBranchResult
v3-3-testPR Link

github-actionsBot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jun 28, 2026
…tion, not inline credentials (apache#68373)
* Document that Dag bundle kwargs should reference a Connection, not inline credentials
Generated-by: Claude Opus 4.8 (1M context) following the guidelines at
https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions
* Fix docs spelling: reword 'inlined' (sphinx spellcheck)
(cherry picked from commit 8c74f8e)
Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Generated-by: Claude Opus 4.8 (1M context)
aws-airflow-bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Jun 28, 2026
…tion, not inline credentials (apache#68373)
* Document that Dag bundle kwargs should reference a Connection, not inline credentials
Generated-by: Claude Opus 4.8 (1M context) following the guidelines at
https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions
* Fix docs spelling: reword 'inlined' (sphinx spellcheck)
(cherry picked from commit 8c74f8e)
Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Generated-by: Claude Opus 4.8 (1M context)
vatsrahul1001 pushed a commit that referenced this pull request Jun 28, 2026
…tion, not inline credentials (#68373) (#69105)
* Document that Dag bundle kwargs should reference a Connection, not inline credentials
Generated-by: Claude Opus 4.8 (1M context) following the guidelines at
https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions
* Fix docs spelling: reword 'inlined' (sphinx spellcheck)
(cherry picked from commit 8c74f8e)
Generated-by: Claude Opus 4.8 (1M context)
Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
karenbraganz pushed a commit to karenbraganz/airflow that referenced this pull request Jun 30, 2026
…line credentials (apache#68373)
* Document that Dag bundle kwargs should reference a Connection, not inline credentials
Generated-by: Claude Opus 4.8 (1M context) following the guidelines at
https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions
* Fix docs spelling: reword 'inlined' (sphinx spellcheck)
Generated-by: Claude Opus 4.8 (1M context)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@potiuk@vatsrahul1001