apache/airflow

Import error of new dags cannot be shown on web UI for common users

Open

#29,897 opened on 2023年3月3日

GitHub で見る
 (23 comments) (0 reactions) (0 assignees)Python (44,809 stars) (16,781 forks)batch import
_eol_affected_version:2.6area:coregood first issuekind:bug

説明

Apache Airflow version

2.5.1

What happened

When we put a new dag in airflow and the dag contains some error which cannot be parsed by the airflow scheduler,

  • The common role user is assigned CommRole which has the permission of read Import Error image

  • Expectation: we expect Import Error should be seen in ui by the common user assigned CommRole (controlled by access_control)

  • Result : We cannot see the import error by common Role user(only admin can see it )

  • admin see this: image
  • common user see no dag or import error

What you think should happen instead

No response

How to reproduce

Just use a error new dag which grants the permission to common user not a admin

Operating System

Centos7

Versions of Apache Airflow Providers

2.5.0

Deployment

Docker-Compose

Deployment details

No response

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

コントリビューターガイド

技術スタック
pythonreactflask
領域
frontendbackendsecurity
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
active
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Basic understanding of Airflow permissionsFamiliarity with Flask/Python backend
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60
調査方針
Investigate the web UI permission checks for the Import Error view. The likely fix involves updating the permission logic in the relevant Flask views or API endpoints to grant 'read Import Error' permission to common users. Check comments in the issue for suggested code changes, particularly in 'airflow/www/views.py' or similar files.