Skip to content

Align key/id path validation for variables and connections#63897

Open
henry3260 wants to merge 1 commit intoapache:mainfrom
henry3260:align-key-id-in-var-conn
Open

Align key/id path validation for variables and connections#63897
henry3260 wants to merge 1 commit intoapache:mainfrom
henry3260:align-key-id-in-var-conn

Conversation

@henry3260
Copy link
Contributor

Why

Before this PR, similar key/id parameters were handled differently:

  • xcoms.py used Path(min_length=1) (request validation layer),
  • variables.py used Path() plus manual if not variable_key checks (runtime layer),
  • connections.py used Path() without a minimum-length constraint.

This inconsistency caused uneven behavior and error handling for the same class of inputs.

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:task-sdk labels Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:task-sdk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant