You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "zach-overflow (via GitHub)" <gi...@apache.org> on 2026/10/05 13:37:09 UTC

[PR] Migrate airflow-core to `httpx2` [airflow]

zach-overflow opened a new pull request, #74244:
URL: https://github.com/apache/airflow/pull/74244

    <!-- SPDX-License-Identifier: Apache-2.0
         https://www.apache.org/licenses/LICENSE-2.0 -->
   
   
   
   ---
   
   ## What?
   * Migrates `airflow-core`'s usage of `httpx` to `httpx2`, following the general migration guidance [here](https://pydantic.dev/docs/httpx2/get-started/migration/).
   * Stacked off the related task-sdk migration PR (https://github.com/apache/airflow/pull/73116) since there is some code overlap.
   * Adds an [`ast-grep`](https://github.com/ast-grep/ast-grep) pre-commit check to prevent re-introducing `httpx` in the Task SDK, mirroring the providers one.
   
   ## Why?
   * closes: https://github.com/apache/airflow/issues/70522
   * Stacked on: https://github.com/apache/airflow/pull/73116
   
   ##### Was generative AI tooling used to co-author this PR?
   
   <!--
   If generative AI tooling has been used in the process of authoring this PR, please
   change below checkbox to `[X]` followed by the name of the tool, uncomment the "Generated-by".
   -->
   
   - [X] Yes (tool specified below)
   
   Code changes Generated-by: [Claude Code, Opus extra-high effort] following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   Self-reviewed and adjusted as needed by hand
   PR description hand-written
   
   ## Additional Notes
   A similar change to `providers` is already merged. See https://github.com/apache/airflow/pull/72111
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Migrate airflow-core to `httpx2` [airflow]

Posted by "ashb (via GitHub)" <gi...@apache.org>.
ashb merged PR #74244:
URL: https://github.com/apache/airflow/pull/74244


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Migrate airflow-core to `httpx2` [airflow]

Posted by "ashb (via GitHub)" <gi...@apache.org>.
ashb commented on PR #74244:
URL: https://github.com/apache/airflow/pull/74244#issuecomment-6200093726

   Do we need an ast-grep, Ruff has banned imports (I feel I asked this already somewhere else?)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Migrate airflow-core to `httpx2` [airflow]

Posted by "ashb (via GitHub)" <gi...@apache.org>.
ashb commented on PR #74244:
URL: https://github.com/apache/airflow/pull/74244#issuecomment-6345705588

   @vatsrahul1001 I think we should probably merge this and include it in 3.4.0rc1. Any questions or objections?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Migrate airflow-core to `httpx2` [airflow]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #74244:
URL: https://github.com/apache/airflow/pull/74244#issuecomment-6348832812

   ### Backport successfully created: v3-4-test
   
   Note: As of [Merging PRs targeted for Airflow 3.X](https://github.com/apache/airflow/blob/main/dev/README_AIRFLOW3_DEV.md#merging-prs-targeted-for-airflow-3x)
   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](https://apache-airflow.slack.com/archives/C03G9H97MM2) Slack channel.
   
   <table>
           <tr>
           <th>Status</th>
           <th>Branch</th>
           <th>Result</th>
       </tr>
       <tr>
           <td>✅</td>
           <td>v3-4-test</td>
           <td><a href="https://github.com/apache/airflow/pull/75424"><img src="https://img.shields.io/badge/PR-75424-blue" alt="PR Link"></a></td>
       </tr>
   </table>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Migrate airflow-core to `httpx2` [airflow]

Posted by "zach-overflow (via GitHub)" <gi...@apache.org>.
zach-overflow commented on PR #74244:
URL: https://github.com/apache/airflow/pull/74244#issuecomment-6177702971

   This may require some coordination with https://github.com/apache/airflow/pull/74728


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Migrate airflow-core to `httpx2` [airflow]

Posted by "zach-overflow (via GitHub)" <gi...@apache.org>.
zach-overflow commented on PR #74244:
URL: https://github.com/apache/airflow/pull/74244#issuecomment-6201919966

   > Do we need an ast-grep, Ruff has banned imports (I feel I asked this already somewhere else?)
   
   @ashb Ah, yeah you're right that was on my separate PR for migrating the task-sdk from `httpx` &rarr; `httpx2`, and I forgot to take that into account on this PR which I had stacked off the task-sdk PR early on. Will do the same replace with the ruff `banned-imports` alternative 👍.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org