Vasu-Madaan opened a new pull request, #68348:
URL: https://github.com/apache/airflow/pull/68348

   ## Summary
   
   `Log.owner_display_name` is stored in the database but was never included in 
`EventLogResponse`, making it inaccessible via the public API and invisible in 
the Audit Log UI.
   
   This PR:
   - Adds `owner_display_name: str | None` to `EventLogResponse` (direct column 
on `Log`, always present — nullable)
   - Updates the generated OpenAPI YAML and TypeScript types accordingly 
(required, typed as `string | null` to match `owner`)
   - Makes the Audit Log "User" column display `owner_display_name` when 
present, falling back to `owner` when it is `null`
   
   The UI fallback is safe: `owner_display_name ?? owner` — if both are `null` 
the cell is blank (existing behavior for unset owners).
   
   ### Security note
   
   Same access control as `owner` (gated by `requires_access_event_log("GET")` 
plus the per-DAG audit log permission), same trust boundary (both populated by 
the auth manager via `Log.__init__`), same rendering path (React escapes the 
cell content). Follows the existing `dag_display_name` / `task_display_name` 
precedent.
   
   closes: #68334
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (claude-opus-4-7)
   
   Generated-by: Claude Code (claude-opus-4-7) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to