Skip to content

Conversation

@rawwar
Copy link
Contributor

@rawwar rawwar commented Feb 28, 2025

related to #42360

@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Feb 28, 2025
Copy link
Member

@jason810496 jason810496 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Only the test case naming convention also mentioned in #47194 .

@rawwar rawwar marked this pull request as ready for review February 28, 2025 13:20
@rawwar rawwar force-pushed the kalyan/AIP-84/auth/pool branch from b31b544 to 3d23b5e Compare March 3, 2025 10:39
@rawwar rawwar requested a review from pierrejeambrun March 3, 2025 12:41
@pierrejeambrun pierrejeambrun merged commit 3a7246f into apache:main Mar 3, 2025
43 checks passed
shahar1 pushed a commit to shahar1/airflow that referenced this pull request Mar 5, 2025
* add auth to pools

* add 403 tests

* fix

* fix tests

* use Request in requires_access_pool

* fix
@Lee-W Lee-W mentioned this pull request Mar 6, 2025
47 tasks
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
* add auth to pools

* add 403 tests

* fix

* fix tests

* use Request in requires_access_pool

* fix
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:UI Related to UI/UX. For Frontend Developers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants