The GitHub Actions job "Tests (AMD)" on airflow.git/v3-2-test has succeeded.
Run started by GitHub user vatsrahul1001 (triggered by vatsrahul1001).
Head commit for run:
5148057b1b4a22b3b12880a26bc0843fe279c080 / github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
[v3-2-test] Apply requires_access_event_log to GET /eventLogs list endpoint
(#67185) (#67211)
* Apply requires_access_event_log to GET /eventLogs list endpoint
* Fail closed on non-integer event_log_id; fix list-endpoint test mock
requires_access_event_log silently swallowed ValueError on non-integer
event_log_id and fell through to the generic AUDIT_LOG check. Raise
HTTPException(400) instead — matches the fail-closed pattern used by
requires_access_backfill.
Also fix test_requires_access_event_log_no_path_param_uses_generic_check:
the test mocked request.path_params = {} but left request.query_params
as an auto-created Mock attribute, whose .get("dag_id") returned a Mock
(truthy non-None). requires_access_dag then resolved dag_id to that
Mock and called is_authorized_dag with the wrong DagDetails. Mock both
path_params and query_params as empty dicts.
---------
(cherry picked from commit b28681f154f3e7d4f90055e59a640e5a6ffc3615)
Co-authored-by: Pierre Jeambrun <[email protected]>
Co-authored-by: Rahul Vats <[email protected]>
Co-authored-by: vatsrahul1001 <[email protected]>
Report URL: https://github.com/apache/airflow/actions/runs/26144486804
With regards,
GitHub Actions via GitBox
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
