Skip to content

Use ValueError for conflicting Git connection key arguments - #70538

Merged
Miretpl merged 1 commit into
apache:mainfrom
mitre88:narrow-git-hook-exception
Aug 4, 2026
Merged

Use ValueError for conflicting Git connection key arguments#70538
Miretpl merged 1 commit into
apache:mainfrom
mitre88:narrow-git-hook-exception

Conversation

@mitre88

Copy link
Copy Markdown
Contributor

Part of the ongoing clean-up of broad AirflowException usages (enforced by the check-no-new-airflow-exceptions ratchet), following the pattern of #66279.

GitHook raised AirflowException when a connection provides both key_file and private_key — a plain input-validation conflict, now a ValueError. The known_airflow_exceptions.txt entry for the file drops from 1 to 0, and the existing test asserts the new type.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Fable 5)

Generated-by: Claude Code (Fable 5) following the guidelines

@mitre88
mitre88force-pushed the narrow-git-hook-exception branch from 8db008e to fcaafbeCompareAugust 1, 2026 02:24
@Miretpl
Miretpl merged commit e98b9b3 into apache:mainAug 4, 2026
155 checks passed
dabla pushed a commit to dabla/airflow that referenced this pull request Aug 14, 2026
No logic change. Rebased PR apache#70538.
Co-authored-by: mitre88 <mitre88@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@mitre88@Miretpl