-
Notifications
You must be signed in to change notification settings - Fork 16.8k
Open
Labels
area:APIAirflow's REST/HTTP APIAirflow's REST/HTTP APIarea:UIRelated to UI/UX. For Frontend Developers.Related to UI/UX. For Frontend Developers.good first issue
Milestone
Description
Body
Currently the expose config allows deployment manager to expose also sensitive data - when set to True https://airflow.apache.org/docs/apache-airflow/stable/configurations-ref.html#expose-config. The non-sensitive-only value causes sensitive field masking.
With the discussion mentioned in #59838 we agreed that we should never expose sensitive data over any public API where UI user can authenticate (only via task-sdk API where tasks get dedicated JWT token)
This means that:
- Only
True/Falseshould be expected for expose-config andTruemeans that sensitive fields are masked - We should add fallback - when "non-sensitive-data" is set for the parameter it should be treated as
Trueand deprecation warning should be raised - newsfragment should be added explaining the behaviour change
Committer
- I acknowledge that I am a maintainer/committer of the Apache Airflow project.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
area:APIAirflow's REST/HTTP APIAirflow's REST/HTTP APIarea:UIRelated to UI/UX. For Frontend Developers.Related to UI/UX. For Frontend Developers.good first issue