Vamsi-klu commented on PR #67606: URL: https://github.com/apache/airflow/pull/67606#issuecomment-4584959291
The new generated page says it lists the required permission for every `/api/v2` endpoint, but the extractor currently skips any route that has no `dependencies=` list, and then also skips dependencies that are not `requires_access_*`. That means public stable API routes included at `public/__init__.py` such as `/api/v2/monitor/health`, `/api/v2/version`, `/api/v2/auth/login`, and `/api/v2/auth/logout` are omitted from `api_permissions_ref.rst` entirely. Can we either include these routes with an explicit “no Airflow permission required” style row, or narrow the generated page wording to say it only covers endpoints that declare a permission dependency? As written, the generated reference is incomplete relative to its own “every endpoint” claim. --- Drafted-by: Codex (GPT-5); reviewed by @Vamsi-klu before posting -- 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]
