Skip to content

Conversation

@Lee-W
Copy link
Member

@Lee-W Lee-W commented Mar 12, 2025

related: #42360


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@boring-cyborg boring-cyborg bot added the area:API Airflow's REST/HTTP API label Mar 12, 2025
@Lee-W Lee-W marked this pull request as ready for review March 12, 2025 07:52
@Lee-W Lee-W force-pushed the AIP-84-execution-asset branch from fff6ebe to 3d958ad Compare March 12, 2025 08:04
@Lee-W Lee-W mentioned this pull request Mar 12, 2025
47 tasks
@Lee-W Lee-W force-pushed the AIP-84-execution-asset branch from 3d958ad to 3ac675d Compare March 12, 2025 08:53
@Lee-W Lee-W force-pushed the AIP-84-execution-asset branch from 3ac675d to d9d5780 Compare March 12, 2025 11:00
@pierrejeambrun
Copy link
Member

pierrejeambrun commented Mar 12, 2025

After discussion permissions for execution API are not required yet and will not work in the same way when we do. For now we will just need to ensure that the client is authenticated with something like requires_authenticated that just validate the token integrity. I think that falls into @ashb current work around auth + execution api: #46981 (and follow up PRs)

I think we should close this PR and related ones on the execution api.

@ashb
Copy link
Member

ashb commented Mar 12, 2025

Yes, don't do any auth in the Execution API app please. That will all be handled differently to the CoreAPI app.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants